SonarQube C#
  1. SonarQube C#
  2. SONARCS-254

Rule: Avoid a given keyword in comments

    Details

    • Type: New Feature New Feature
    • Status: Closed Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 2.1
    • Labels:
      None
    • Number of attachments :
      0

      Description

      This ticket only aims to create a rule to detect some regular expression within comments.

      It will not provide any widget.

        Issue Links

          Activity

          Hide
          Dinesh Bolkensteyn added a comment -

          Thanks Alexandre for the implementation.

          I'll take care of this ticket from now on.

          Indeed, it should be covered by our ruling IT, to make sure no regressions are introduced in the future, which I will do.

          Show
          Dinesh Bolkensteyn added a comment - Thanks Alexandre for the implementation. I'll take care of this ticket from now on. Indeed, it should be covered by our ruling IT, to make sure no regressions are introduced in the future, which I will do.
          Hide
          Dinesh Bolkensteyn added a comment -

          The rule should not by default be used to detect TODOs.

          It should by default by a no-operation, and only specific instances of the rule must generate violations.

          Show
          Dinesh Bolkensteyn added a comment - The rule should not by default be used to detect TODOs. It should by default by a no-operation, and only specific instances of the rule must generate violations.
          Hide
          Dinesh Bolkensteyn added a comment -

          Done

          The other commented out LOC check has been renamed to start with 'Comment', so that both checks can be easily found and appear regrouped in the list of rules on the quality profile page.

          Show
          Dinesh Bolkensteyn added a comment - Done The other commented out LOC check has been renamed to start with 'Comment', so that both checks can be easily found and appear regrouped in the list of rules on the quality profile page.
          Hide
          Fabrice Bellingard added a comment -

          Tested!

          Show
          Fabrice Bellingard added a comment - Tested!

            People

            • Assignee:
              Dinesh Bolkensteyn
              Reporter:
              Eric Hartmann
            • Votes:
              1 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: