Merge pull request #331 from boesing/feature/php-cs-fixer-checkstyle

boesing · web-flow · commit 60b7e8cb8666 · 2025-01-17T12:09:24.000+01:00
feature: `php-cs-fixer` with `checkstyle` output forwarded to github pull request
diff --git a/src/tools/phpCsFixer.ts b/src/tools/phpCsFixer.ts
@@ -4,7 +4,7 @@ import {ToolExecutionType} from '../enum/toolExecutionType';
 export const PhpCsFixerTool = {
     executionType : ToolExecutionType.STATIC,
     name          : 'PHP CS Fixer',
-    command       : './vendor/bin/php-cs-fixer fix -v --diff --dry-run',
+    command       : './vendor/bin/php-cs-fixer fix -v --diff --dry-run --format=checkstyle | cs2pr',
     filesToCheck  : [ '.php-cs-fixer.php', '.php-cs-fixer.dist.php' ],
     toolType      : ToolType.CODE_CHECK,
 };
diff --git a/tests/code-check-phpcsfixer-php-dist/matrix.json b/tests/code-check-phpcsfixer-php-dist/matrix.json
@@ -2,7 +2,7 @@
     "include": [
         {
             "name": "PHP CS Fixer [7.4, latest]",
-            "job": "{\"command\":\"./vendor/bin/php-cs-fixer fix -v --diff --dry-run\",\"php\":\"7.4\",\"extensions\":[],\"ini\":[],\"dependencies\":\"latest\",\"ignore_platform_reqs_8\":false,\"ignore_php_platform_requirement\":false,\"additional_composer_arguments\":[],\"before_script\":[]}",
+            "job": "{\"command\":\"./vendor/bin/php-cs-fixer fix -v --diff --dry-run --format=checkstyle | cs2pr\",\"php\":\"7.4\",\"extensions\":[],\"ini\":[],\"dependencies\":\"latest\",\"ignore_platform_reqs_8\":false,\"ignore_php_platform_requirement\":false,\"additional_composer_arguments\":[],\"before_script\":[]}",
             "operatingSystem": "ubuntu-latest",
             "action": "laminas/laminas-continuous-integration-action@v1"
         }
diff --git a/tests/code-check-phpcsfixer-php/matrix.json b/tests/code-check-phpcsfixer-php/matrix.json
@@ -2,7 +2,7 @@
     "include": [
         {
             "name": "PHP CS Fixer [7.4, latest]",
-            "job": "{\"command\":\"./vendor/bin/php-cs-fixer fix -v --diff --dry-run\",\"php\":\"7.4\",\"extensions\":[],\"ini\":[],\"dependencies\":\"latest\",\"ignore_platform_reqs_8\":false,\"ignore_php_platform_requirement\":false,\"additional_composer_arguments\":[],\"before_script\":[]}",
+            "job": "{\"command\":\"./vendor/bin/php-cs-fixer fix -v --diff --dry-run --format=checkstyle | cs2pr\",\"php\":\"7.4\",\"extensions\":[],\"ini\":[],\"dependencies\":\"latest\",\"ignore_platform_reqs_8\":false,\"ignore_php_platform_requirement\":false,\"additional_composer_arguments\":[],\"before_script\":[]}",
             "operatingSystem": "ubuntu-latest",
             "action": "laminas/laminas-continuous-integration-action@v1"
         }

Original file line number	Diff line number	Diff line change
`@@ -2,7 +2,7 @@`
`2`	`2`	`"include": [`
`3`	`3`	`{`
`4`	`4`	`"name": "PHP CS Fixer [7.4, latest]",`
`5`		`- "job": "{\"command\":\"./vendor/bin/php-cs-fixer fix -v --diff --dry-run\",\"php\":\"7.4\",\"extensions\":[],\"ini\":[],\"dependencies\":\"latest\",\"ignore_platform_reqs_8\":false,\"ignore_php_platform_requirement\":false,\"additional_composer_arguments\":[],\"before_script\":[]}",`
	`5`	`+ "job": "{\"command\":\"./vendor/bin/php-cs-fixer fix -v --diff --dry-run --format=checkstyle \| cs2pr\",\"php\":\"7.4\",\"extensions\":[],\"ini\":[],\"dependencies\":\"latest\",\"ignore_platform_reqs_8\":false,\"ignore_php_platform_requirement\":false,\"additional_composer_arguments\":[],\"before_script\":[]}",`
`6`	`6`	`"operatingSystem": "ubuntu-latest",`
`7`	`7`	`"action": "laminas/laminas-continuous-integration-action@v1"`
`8`	`8`	`}`