Update reviewdog configs, and improve Unity logging (#195)

* feat: update GitHub Actions permissions for pull requests

* fix: adjust permissions configuration in workflow files

* refactor: replace Console.WriteLine with Debug.Log

* feat: update permissions in GitHub Actions workflow

* fix: adjust reviewdog filter mode in lint workflow

* fix: update default exit code and remove unused permissions

* feat: add exclude paths to Unity CS linter workflow

Author: IShix-g

.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

@@ -32,13 +32,16 @@ on:
         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

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

@@ -22,12 +22,6 @@ on:
         type: string
         default: ''
 
-permissions:
-  contents: write
-  actions: read
-  checks: write
-  statuses: write
-
 jobs:
   unity-cs-linter:
     uses: ./.github/workflows/reusable-unity-cs-linter-dispatch.yaml

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

@@ -12,4 +12,5 @@ jobs:
       head-branch: ${{ github.event.pull_request.head.ref }}
       base-branch: ${{ github.event.pull_request.base.ref }}
       editorconfig-branch: ${{ github.event.repository.default_branch }}
-      include-paths: 'Assets/Tests/StyleTest/.*'
+      include-paths: 'Assets/Tests/StyleTest/.*'
+      exclude-paths: 'Assets/Tests/StyleTest/FileNameIssueStyleViolationTest.cs'

Assets/Tests/StyleTest/FileNameIssueStyleViolationTest.cs

@@ -1,4 +1,5 @@
-using System;
+
+using UnityEngine;
 
 namespace CodeStyleExample
 {
@@ -17,28 +18,28 @@ public class FileNameIssueStyleViolationTest
         // メソッド名の命名規則違反（PascalCaseが推奨される）
         public void printmessage() // 【違反】PascalCaseでない
         {
-            Console.WriteLine("This should follow PascalCase.");
+            Debug.Log("This should follow PascalCase.");
         }
 
         // 正しいメソッド
         public void DisplayMessage()
         {
-            Console.WriteLine(_exampleField);
+            Debug.Log(_exampleField);
         }
 
         // if文に波括弧{}が無い（スタイル違反: IDE0011）
         public void CheckCondition(int value)
         {
             if (value > 0) // 【違反】波括弧が無い
-                Console.WriteLine("Value is positive.");
+                Debug.Log("Value is positive.");
         }
 
         // 正しい条件分岐
         public void CorrectConditionCheck(int value)
         {
             if (value > 0)
             {
-                Console.WriteLine("Value is positive.");
+                Debug.Log("Value is positive.");
             }
         }