Unity Lint Test

.github/workflows/reusable-lint-cs-files.yaml

@@ -203,7 +203,7 @@ jobs:
           reviewdog -f=sarif \
             -name="StyleCop" \
             -reporter=${{ inputs.review-reporter }} \
-            -filter-mode=file \
+            -filter-mode=nofilter \
             -level=${{ inputs.review-level }} \
             -fail-level=${{ inputs.review-exit-code }} \
             < output.sarif

.github/workflows/reusable-unity-cs-linter-dispatch.yaml

@@ -29,16 +29,19 @@ on:
         type: string
         default: 'warning'
       review-exit-code:
-        description: 'Exit code string based on reviewdog exit codes. [any,info,warning,error] https://github.com/reviewdog/reviewdog?tab=readme-ov-file#exit-codes'
+        description: 'Exit code string based on reviewdog exit codes. [none,any,info,warning,error] https://github.com/reviewdog/reviewdog?tab=readme-ov-file#exit-codes'
         required: false
         type: string
-        default: 'error'
+        default: 'none'
       max-log-lines:
         description: 'Maximum number of lines to print from the log'
         required: false
         type: string
         default: '50'
 
+permissions:
+  checks: write
+
 jobs:
   branch-conf:
     runs-on: ubuntu-22.04
@@ -155,4 +158,7 @@ jobs:
       ref: ${{ needs.branch-conf.outputs.head-branch || needs.branch-conf.outputs.base-branch }}
       editorconfig-branch: ${{ needs.branch-conf.outputs.editorconfig-branch }}
       files: ${{ needs.detect-cs-files.outputs.files }}
-      review-reporter: 'github-check'
+      max-log-lines: ${{ inputs.max-log-lines }}
+      review-reporter: 'github-check'
+      review-level: ${{ inputs.review-level }}
+      review-exit-code: ${{ inputs.review-exit-code }}

.github/workflows/reusable-unity-cs-linter-pull-request.yaml

@@ -33,7 +33,7 @@ on:
         type: string
         default: 'warning'
       review-exit-code:
-        description: 'Exit code string based on reviewdog exit codes. [any,info,warning,error] https://github.com/reviewdog/reviewdog?tab=readme-ov-file#exit-codes'
+        description: 'Exit code string based on reviewdog exit codes. [none,any,info,warning,error] https://github.com/reviewdog/reviewdog?tab=readme-ov-file#exit-codes'
         required: false
         type: string
         default: 'error'
@@ -166,4 +166,7 @@ jobs:
       ref: ${{ needs.branch-conf.outputs.head-branch || needs.branch-conf.outputs.base-branch }}
       editorconfig-branch: ${{ needs.branch-conf.outputs.editorconfig-branch }}
       files: ${{ needs.detect-cs-files.outputs.files }}
-      review-reporter: 'github-pr-review'
+      max-log-lines: ${{ inputs.max-log-lines }}
+      review-reporter: 'github-pr-review'
+      review-level: ${{ inputs.review-level }}
+      review-exit-code: ${{ inputs.review-exit-code }}

.github/workflows/reusable-unity-test.yaml

@@ -38,6 +38,9 @@ on:
       UNITY_SERIAL:
         required: false
 
+permissions:
+  checks: write
+
 jobs:
   print-unity-version:
     runs-on: ubuntu-22.04

.github/workflows/test-unity-cs-linter-dispatch.yaml

@@ -1,13 +1,13 @@
-name: (Test) Unity C# Linter
+name: (Test) Unity C# Linter for Dispatch
 
 on:
   workflow_dispatch:
     inputs:
       include-paths:
         description: 'Comma-separated regex patterns for files or directories to include. More info: https://www.gnu.org/software/grep/manual/'
-        required: false
+        required: true
         type: string
-        default: 'Assets/.*'
+        default: 'Assets/Tests/StyleTest/FileNameIssueStyleViolationTest.cs'
       exclude-paths:
         description: 'Exclude files or directories (regex, comma-separated)'
         required: false
@@ -17,7 +17,7 @@ on:
     inputs:
       include-paths:
         type: string
-        default: 'Assets/.*'
+        default: 'Assets/Tests/StyleTest/FileNameIssueStyleViolationTest.cs'
       exclude-paths:
         type: string
         default: ''

Assets/Tests/StyleTest/FileNameIssueStyleViolationTest.cs

@@ -0,0 +1,54 @@
+
+using UnityEngine;
+
+namespace CodeStyleExample
+{
+    // クラス名はPascalCaseが推奨されているが、lowercase名を使った例（スタイル違反）
+    public class FileNameIssueStyleViolationTest
+    {
+        // メンバー変数の命名規則違反（推奨: _camelCase または PascalCase）
+        private int TestVariable = 10;
+
+        // 正しい名前のフィールド
+        private string _exampleField = "Valid";
+
+        // Staticメンバー配置順序の違反（推奨: 静的要素 → インスタンスメンバーの順序）
+        public static readonly string StaticExample = "Static Value";
+
+        // メソッド名の命名規則違反（PascalCaseが推奨される）
+        public void printmessage() // 【違反】PascalCaseでない
+        {
+            Debug.Log("This should follow PascalCase.");
+        }
+
+        // 正しいメソッド
+        public void DisplayMessage()
+        {
+            Debug.Log(_exampleField);
+        }
+
+        // if文に波括弧{}が無い（スタイル違反: IDE0011）
+        public void CheckCondition(int value)
+        {
+            if (value > 0) // 【違反】波括弧が無い
+                Debug.Log("Value is positive.");
+        }
+
+        // 正しい条件分岐
+        public void CorrectConditionCheck(int value)
+        {
+            if (value > 0)
+            {
+                Debug.Log("Value is positive.");
+            }
+        }
+
+        // プロパティ定義ではPascalCaseが推奨
+        public int myProperty { get; set; } = 42; // 【違反】PascalCaseでない名前
+
+        // 正しいプロパティ定義
+        public int MyProperty { get; set; } = 100;
+
+        // 最後に改行がない（スタイル違反: `insert_final_newline=true`）
+    }
+}

Assets/Tests/StyleTest/StyleTest2.cs

@@ -0,0 +1,15 @@
+namespace Tests.StyleTest
+{
+    public sealed class StyleTest2
+    {
+        static string s_staticField = string.Empty;
+        string test;
+        string _test2;
+
+        void Test()
+        {
+            var test = this._test2;
+
+        }
+    }
+}

Assets/Tests/StyleTest/StyleTest3.cs

@@ -0,0 +1,14 @@
+namespace Tests.StyleTest
+{
+    public sealed class StyleTest3
+    {
+        public static readonly string TestStaticReadonly = "TestStaticReadonly";
+        static string s_staticField = string.Empty;
+        string _test;
+        string _test2;
+
+        void Test()
+        {
+        }
+    }
+}

Assets/Tests/StyleTest/StyleTest4.cs

@@ -0,0 +1,15 @@
+namespace Tests.StyleTest
+{
+    public sealed class StyleTest4
+    {
+        static string s_staticField = string.Empty;
+        string _test;
+        string _test2;
+
+        void Test()
+        {
+            var test = this._test2;
+
+        }
+    }
+}