Skip to content

Commit

Permalink
DX: Improve Mess Detector Integration (#7224)
Browse files Browse the repository at this point in the history
  • Loading branch information
Wirone authored Aug 11, 2023
1 parent 63da145 commit d261c82
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 4 deletions.
6 changes: 3 additions & 3 deletions .github/workflows/sca.yml
Original file line number Diff line number Diff line change
Expand Up @@ -117,13 +117,13 @@ jobs:
echo 'EOF' >> $GITHUB_ENV
fi
- name: Check - phpmd
- name: Check - Mess Detector (phpmd)
if: ${{ env.CHANGED_PHP_FILES }}
run: |
if [ '${{ github.event_name }}' == 'pull_request' ]; then
./dev-tools/vendor/bin/phpmd `echo "$CHANGED_PHP_FILES" | grep -v /Fixtures/ | xargs | sed 's/ /,/g'` github phpmd.xml
./dev-tools/vendor/bin/phpmd `echo "$CHANGED_PHP_FILES" | grep -v /Fixtures/ | xargs | sed 's/ /,/g'` github ./dev-tools/mess-detector/phpmd.xml
else
./dev-tools/vendor/bin/phpmd `echo "$CHANGED_PHP_FILES" | grep -v /Fixtures/ | xargs | sed 's/ /,/g'` ansi phpmd.xml
./dev-tools/vendor/bin/phpmd `echo "$CHANGED_PHP_FILES" | grep -v /Fixtures/ | xargs | sed 's/ /,/g'` ansi ./dev-tools/mess-detector/phpmd.xml
fi
- name: Check - ensure test files are not present in the archive
Expand Down
2 changes: 1 addition & 1 deletion composer.json
Original file line number Diff line number Diff line change
Expand Up @@ -89,7 +89,7 @@
"dev-tools:docs": "@php dev-tools/doc.php",
"dev-tools:install": "@composer -d dev-tools install",
"docs": "@dev-tools:docs",
"mess-detector": "@php dev-tools/vendor/bin/phpmd . ansi phpmd.xml",
"mess-detector": "@php dev-tools/vendor/bin/phpmd . ansi dev-tools/mess-detector/phpmd.xml --exclude vendor/*,dev-tools/vendor/*,dev-tools/phpstan/*,tests/Fixtures/*",
"phpstan": "@php -d memory_limit=256M dev-tools/vendor/bin/phpstan analyse",
"phpstan:baseline": "@php -d memory_limit=256M dev-tools/vendor/bin/phpstan analyse --generate-baseline=./dev-tools/phpstan/baseline.php",
"qa": "@quality-assurance",
Expand Down
File renamed without changes.

0 comments on commit d261c82

Please sign in to comment.