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

Rule S00100 : Method name must comply with a naming convention

    Details

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

      Activity

      Show
      Dinesh Bolkensteyn added a comment - Default regexp from http://msdn.microsoft.com/en-us/library/4df752aw(v=vs.71).aspx
      Hide
      Dinesh Bolkensteyn added a comment - - edited

      Let's verify that event handlers are excluded from this rule.

      Show
      Dinesh Bolkensteyn added a comment - - edited Let's verify that event handlers are excluded from this rule.
      Hide
      Dinesh Bolkensteyn added a comment -

      As the class name, this rule now also reports the violation on the line on which the method name occurs:

      void  // Previously, the violation was reported here
      foo() // But now, it's reported here! Way better!
      {
      }
      
      Show
      Dinesh Bolkensteyn added a comment - As the class name, this rule now also reports the violation on the line on which the method name occurs: void // Previously, the violation was reported here foo() // But now, it's reported here! Way better! { }
      Hide
      Fabrice Bellingard added a comment -

      Digits should be allowed by default. See http://msdn.microsoft.com/en-us/library/vstudio/ms229045%28v=vs.100%29.aspx: "Do not use underscores, hyphens, or any other nonalphanumeric characters".

      Show
      Fabrice Bellingard added a comment - Digits should be allowed by default. See http://msdn.microsoft.com/en-us/library/vstudio/ms229045%28v=vs.100%29.aspx: "Do not use underscores, hyphens, or any other nonalphanumeric characters".
      Hide
      Fabrice Bellingard added a comment -

      Also, there was a typo in the rule description (fixed on the rule spec repo): "This rule allow*s* ..."

      Show
      Fabrice Bellingard added a comment - Also, there was a typo in the rule description (fixed on the rule spec repo): "This rule allow*s* ..."
      Hide
      Dinesh Bolkensteyn added a comment -

      Done

      Show
      Dinesh Bolkensteyn added a comment - Done
      Hide
      Fabrice Bellingard added a comment -

      Tested!

      Show
      Fabrice Bellingard added a comment - Tested!

        People

        • Assignee:
          Dinesh Bolkensteyn
          Reporter:
          Freddy Mallet
        • Votes:
          0 Vote for this issue
          Watchers:
          3 Start watching this issue

          Dates

          • Created:
            Updated:
            Resolved: