Skip to content
Closed
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
42 commits
Select commit Hold shift + click to select a range
7f8adac
feat: add style compliance test script
IShix-g Feb 22, 2025
fdc8dc4
refactor: remove unused interface from StyleTestScript
IShix-g Feb 22, 2025
14f8cba
Merge branch 'feature/csharp-lint' into feature/csharp-lint-test2
IShix-g Feb 22, 2025
17d8146
Merge branch 'feature/csharp-lint' into feature/csharp-lint-test2
IShix-g Feb 22, 2025
08f9645
feat: add placeholder method in StyleTestScript
IShix-g Feb 22, 2025
c950162
Merge branch 'feature/csharp-lint' into feature/csharp-lint-test2
IShix-g Feb 22, 2025
8b0cebc
refactor: fix code formatting in StyleTestScript.cs
IShix-g Feb 22, 2025
bbf04d4
Merge branch 'feature/csharp-lint' into feature/csharp-lint-test2
IShix-g Feb 22, 2025
ca27d27
refactor: adjust field indentation in StyleTestScript
IShix-g Feb 22, 2025
d9d8b32
Merge branch 'feature/csharp-lint' into feature/csharp-lint-test2
IShix-g Feb 22, 2025
d75f499
refactor: fix code formatting in StyleTestScript
IShix-g Feb 22, 2025
9334aac
Merge branch 'feature/csharp-lint' into feature/csharp-lint-test2
IShix-g Feb 22, 2025
4a6b342
refactor: fix code formatting in StyleTestScript.cs
IShix-g Feb 22, 2025
92bbf64
Merge branch 'feature/csharp-lint' into feature/csharp-lint-test2
IShix-g Feb 22, 2025
407d0e1
refactor: fix code formatting in StyleTestScript.cs
IShix-g Feb 22, 2025
1187bda
feat: add StyleTestScript2 for testing code style rules
IShix-g Feb 22, 2025
533fc74
refactor: fix inconsistent indentation in code
IShix-g Feb 23, 2025
44a3fb6
refactor: fix indentation in StyleTestScript2.cs
IShix-g Feb 23, 2025
5d4aa64
refactor: fix code formatting in StyleTestScript2 functionality was
IShix-g Feb 23, 2025
c0b5366
Merge branch 'feature/csharp-lint' into feature/csharp-lint-test2
IShix-g Feb 23, 2025
b1b2016
Merge branch 'feature/csharp-lint' into feature/csharp-lint-test2
IShix-g Feb 23, 2025
2932a63
refactor: fix incorrect field indentation in StyleTestScript2
IShix-g Feb 23, 2025
d6dfd69
Merge branch 'feature/csharp-lint' into feature/csharp-lint-test2
IShix-g Feb 23, 2025
9d4ad5e
refactor: fix code indentation in StyleTestScript2.cs
IShix-g Feb 23, 2025
0e30595
Merge branch 'feature/csharp-lint' into feature/csharp-lint-test2
IShix-g Feb 23, 2025
2217ad7
refactor: fix inconsistent indentation in StyleTestScript2
IShix-g Feb 23, 2025
0e2d6e4
refactor: apply code style adjustments for consistency
IShix-g Feb 24, 2025
0cea84b
refactor: rename interface to follow naming conventions
IShix-g Feb 24, 2025
b6f9cd0
Merge branch 'feature/csharp-lint' into feature/csharp-lint-test2
IShix-g Feb 24, 2025
24a48e0
fix: correct unintended whitespace in field declaration
IShix-g Feb 24, 2025
b7d2440
Merge branch 'feature/csharp-lint' into feature/csharp-lint-test2
IShix-g Feb 24, 2025
8259fbb
refactor: fix code formatting issues in StyleTestScript
IShix-g Feb 24, 2025
216da9e
refactor: adjust code formatting and remove extra newline
IShix-g Feb 24, 2025
d097da2
refactor: improve code formatting and remove redundant comments
IShix-g Feb 24, 2025
3959c20
Merge branch 'feature/csharp-lint' into feature/csharp-lint-test2
IShix-g Feb 24, 2025
a2dacb9
refactor: reorder fields and fix spacing in interface
IShix-g Feb 24, 2025
0d05265
Merge branch 'feature/csharp-lint' into feature/csharp-lint-test2
IShix-g Feb 24, 2025
8da6657
refactor: improve naming conventions and code style
IShix-g Feb 24, 2025
0bcc3bf
Merge branch 'feature/csharp-lint' into feature/csharp-lint-test2
IShix-g Feb 24, 2025
e8356ee
refactor: clean up code style and remove redundant method
IShix-g Feb 24, 2025
62c4afb
Merge branch 'feature/csharp-lint' into feature/csharp-lint-test2
IShix-g Feb 24, 2025
aee1716
refactor: improve code style and naming consistency
IShix-g Feb 24, 2025
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 3 additions & 0 deletions Assets/Tests/StyleTest.meta

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

42 changes: 42 additions & 0 deletions Assets/Tests/StyleTest/StyleTestScript.cs
Original file line number Diff line number Diff line change
@@ -0,0 +1,42 @@

using UnityEngine;

namespace Tests.StyleTest
{
public sealed class StyleTestScript
{
public int CorrectProperty { get; set; }

int _incorrect_field;
string stringField;

Check failure on line 11 in Assets/Tests/StyleTest/StyleTestScript.cs

View workflow job for this annotation

GitHub Actions / release-notes / lint-cs-files

Naming rule violation: Missing prefix: '_'
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 CheckRules()
{
if (CorrectProperty > 0)
{
Debug.Log("Correct Property is greater than zero");
}

if (CorrectProperty > 0)

Check warning on line 20 in Assets/Tests/StyleTest/StyleTestScript.cs

View workflow job for this annotation

GitHub Actions / release-notes / lint-cs-files

Add braces to 'if' statement.
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.

Debug.Log("Correct Property is greater than zero");

Check warning on line 21 in Assets/Tests/StyleTest/StyleTestScript.cs

View workflow job for this annotation

GitHub Actions / release-notes / lint-cs-files

Braces should not be omitted
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

}

public void VariableDeclaration()
{
var number = 5;
int specificNumber = 10;
}

public void Methord()
{

}

void PrivateMethod()
{

}
}

public interface IStyleTestInterface { }
}
3 changes: 3 additions & 0 deletions Assets/Tests/StyleTest/StyleTestScript.cs.meta

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

28 changes: 28 additions & 0 deletions Assets/Tests/StyleTest/StyleTestScript2.cs
Original file line number Diff line number Diff line change
@@ -0,0 +1,28 @@
namespace Tests.StyleTest
{
public sealed class StyleTestScript2
{
public static readonly string StaticProperty = "This is a static property";
static readonly string staticField = "This is a static field";

Check failure on line 6 in Assets/Tests/StyleTest/StyleTestScript2.cs

View workflow job for this annotation

GitHub Actions / release-notes / lint-cs-files

Naming rule violation: Missing prefix: 's_'
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: 's_'

static string staticField2 = "This is a static field";

Check failure on line 7 in Assets/Tests/StyleTest/StyleTestScript2.cs

View workflow job for this annotation

GitHub Actions / release-notes / lint-cs-files

Naming rule violation: Missing prefix: 's_'
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: 's_'

public int CorrectProperty { get; set; }

Check warning on line 8 in Assets/Tests/StyleTest/StyleTestScript2.cs

View workflow job for this annotation

GitHub Actions / release-notes / lint-cs-files

Elements should be separated by blank line
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 🐶
Elements should be separated by blank line


private int _incorrectField;
string _stringField;

public void CheckRules()
{

}

public void variableDeclaration()

Check failure on line 18 in Assets/Tests/StyleTest/StyleTestScript2.cs

View workflow job for this annotation

GitHub Actions / release-notes / lint-cs-files

Naming rule violation: These words must begin with upper case characters: variableDeclaration
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: variableDeclaration

{

}

internal void Methord()
{

}
}
}
3 changes: 3 additions & 0 deletions Assets/Tests/StyleTest/StyleTestScript2.cs.meta

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Loading