Merge pull request #918 from cakephp/ADmad-patch-1

Fix CS errors display on PRs

.github/workflows/ci.yml

@@ -168,7 +168,7 @@ jobs:
       run: composer stan-setup
 
     - name: Run phpcs
-      run: vendor/bin/phpcs
+      run: vendor/bin/phpcs --report=checkstyle | cs2pr
 
     - name: Run psalm
       if: success() || failure()

composer.json

@@ -50,8 +50,8 @@
             "@test",
             "@cs-check"
         ],
-        "cs-check": "phpcs --parallel=16",
-        "cs-fix": "phpcbf --parallel=16",
+        "cs-check": "phpcs -p --parallel=16",
+        "cs-fix": "phpcbf -p --parallel=16",
         "stan": "phpstan analyse && psalm.phar",
         "stan-setup": "cp composer.json composer.backup && composer require --dev phpstan/phpstan:^1.7 psalm/phar:~4.27.0 && mv composer.backup composer.json",
         "test": "phpunit",

phpcs.xml

@@ -1,7 +1,7 @@
 <?xml version="1.0"?>
 <ruleset name="CakePHP Bake">
     <config name="installed_paths" value="../../cakephp/cakephp-codesniffer" />
-    <arg value="nps"/>
+    <arg value="ns"/>
 
     <file>src/</file>
     <file>tests/</file>