-
Notifications
You must be signed in to change notification settings - Fork 0
feat: add style compliance test script #165
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
| public sealed class StyleTestScript2 | ||
| { | ||
| public static readonly string StaticProperty = "This is a static property"; | ||
| static readonly string staticField = "This is a static field"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🚫 [StyleCop] reported by reviewdog 🐶
Naming rule violation: Missing prefix: 's_'
| { | ||
| public static readonly string StaticProperty = "This is a static property"; | ||
| static readonly string staticField = "This is a static field"; | ||
| static string staticField2 = "This is a static field"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🚫 [StyleCop] reported by reviewdog 🐶
Naming rule violation: Missing prefix: 's_'
|
|
||
| } | ||
|
|
||
| public void variableDeclaration() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🚫 [StyleCop] reported by reviewdog 🐶
Naming rule violation: These words must begin with upper case characters: variableDeclaration
| public static readonly string StaticProperty = "This is a static property"; | ||
| static readonly string staticField = "This is a static field"; | ||
| static string staticField2 = "This is a static field"; | ||
| public int CorrectProperty { get; set; } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Elements should be separated by blank line
| Debug.Log("Correct Property is greater than zero"); | ||
| } | ||
|
|
||
| if (CorrectProperty > 0) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Add braces to 'if' statement.
| public int CorrectProperty { get; set; } | ||
|
|
||
| int _incorrect_field; | ||
| string stringField; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🚫 [StyleCop] reported by reviewdog 🐶
Naming rule violation: Missing prefix: '_'
| } | ||
|
|
||
| if (CorrectProperty > 0) | ||
| Debug.Log("Correct Property is greater than zero"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Braces should not be omitted
No description provided.