-
Couldn't load subscription status.
- Fork 0
Unity Lint Test #187
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
Unity Lint Test #187
Conversation
# Conflicts: # .github/workflows/reusable-unity-cs-linter-dispatch.yaml # .github/workflows/test-unity-cs-linter-dispatch.yaml
| // if文に波括弧{}が無い(スタイル違反: IDE0011) | ||
| public void CheckCondition(int value) | ||
| { | ||
| if (value > 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.
| } | ||
|
|
||
| // プロパティ定義ではPascalCaseが推奨 | ||
| public int myProperty { get; set; } = 42; // 【違反】PascalCaseでない名前 |
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: myProperty
| public class FileNameIssueStyleViolationTest | ||
| { | ||
| // メンバー変数の命名規則違反(推奨: _camelCase または PascalCase) | ||
| private int TestVariable = 10; |
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: '_'
| public static readonly string StaticExample = "Static Value"; | ||
|
|
||
| // メソッド名の命名規則違反(PascalCaseが推奨される) | ||
| public void printmessage() // 【違反】PascalCaseでない |
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: printmessage
| public sealed class StyleTest2 | ||
| { | ||
| static string s_staticField = string.Empty; | ||
| string test; |
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: '_'
| public void CheckCondition(int value) | ||
| { | ||
| if (value > 0) // 【違反】波括弧が無い | ||
| Debug.Log("Value is positive."); |
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.