Fix/ignore phpcs on tests (#6)

Fail non silently. Catch phpcs exit code and exit with that one.

.github/workflows/phpcs.yml

@@ -66,4 +66,24 @@ jobs:
         env:
           REVIEWDOG_GITHUB_API_TOKEN: ${{ secrets.access-token }}
         run: |
-          vendor/bin/phpcs --report=json ${CHANGED_FILES} | jq -r ' .files | to_entries[] | .key as $path | .value.messages[] as $msg | "\($path):\($msg.line):\($msg.column):`\($msg.source)`<br>\($msg.message)" ' | reviewdog -efm="%f:%l:%c:%m" -name="phpcs" -filter-mode="added" -fail-on-error=true -reporter=github-pr-review
+          # Run phpcs and capture both the output and the exit code
+          JSON_REPORT=$(vendor/bin/phpcs --report=json ${{ env.CHANGED_FILES }} || echo "")
+          PHPCS_EXIT_CODE=$?
+
+          # Check if phpcs produced a JSON report
+          if [ -z "$JSON_REPORT" ]; then
+            echo "No JSON report generated by phpcs"
+            exit $PHPCS_EXIT_CODE
+          fi
+
+          # Validate the JSON
+          if ! echo "$JSON_REPORT" | jq empty; then
+            echo "Invalid JSON"
+            exit 1
+          fi
+
+          # Process JSON and run reviewdog
+          echo "$JSON_REPORT" | jq -r ' .files | to_entries[] | .key as $path | .value.messages[] as $msg | "\($path):\($msg.line):\($msg.column):`\($msg.source)`<br>\($msg.message)" ' | reviewdog -efm="%f:%l:%c:%m" -name="phpcs" -filter-mode="added" -fail-on-error=true -reporter=github-pr-review
+
+          # Exit with the original phpcs exit code
+          exit $PHPCS_EXIT_CODE