Skip to content

Conversation

@IShix-g
Copy link
Owner

@IShix-g IShix-g commented Feb 28, 2025

No description provided.

@IShix-g IShix-g changed the title feat: add StyleTest2 for style-related testing Style Test Feb 28, 2025
@IShix-g IShix-g changed the title Style Test Unity Lint Test Feb 28, 2025
IShix-g added 27 commits March 1, 2025 14:05
# 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) // 【違反】波括弧が無い
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ [StyleCop] reported by reviewdog 🐶
Add braces to 'if' statement.

}

// プロパティ定義ではPascalCaseが推奨
public int myProperty { get; set; } = 42; // 【違反】PascalCaseでない名前
Copy link
Contributor

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;
Copy link
Contributor

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でない
Copy link
Contributor

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;
Copy link
Contributor

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.");
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ [StyleCop] reported by reviewdog 🐶
Braces should not be omitted

@IShix-g IShix-g closed this Mar 1, 2025
@IShix-g IShix-g deleted the develop/unity-lint-head branch March 1, 2025 07:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants