Skip to content

Commit

Permalink
Remove conflict with HHVM
Browse files Browse the repository at this point in the history
  • Loading branch information
sanmai committed Nov 26, 2020
1 parent 9488546 commit eb73769
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 6 deletions.
10 changes: 7 additions & 3 deletions .github/workflows/sca.yaml
Expand Up @@ -22,6 +22,8 @@ jobs:
steps:
- name: Checkout code
uses: actions/checkout@v2
with:
fetch-depth: 0

- name: Setup PHP
uses: shivammathur/setup-php@v2
Expand Down Expand Up @@ -77,15 +79,17 @@ jobs:
- name: Find changed files (for pull request)
if: ${{ github.event_name == 'pull_request' }}
run: |
git diff origin/$GITHUB_BASE_REF --name-only --diff-filter=ACMRTUXB | grep -E "\.php$" || true
echo 'CHANGED_PHP_FILES<<EOF' >> $GITHUB_ENV
git diff origin/$GITHUB_BASE_REF --name-only --diff-filter=ACMRTUXB | grep -E "\.php$" >> $GITHUB_ENV
git diff origin/$GITHUB_BASE_REF --name-only --diff-filter=ACMRTUXB | grep -E "\.php$" || true >> $GITHUB_ENV
echo 'EOF' >> $GITHUB_ENV
- name: Find changed files (for push)
if: ${{ github.event_name != 'pull_request' }}
run: |
git diff --name-only --diff-filter=ACMRTUXB HEAD~..HEAD | grep -E "\.php$" || true
echo 'CHANGED_PHP_FILES<<EOF' >> $GITHUB_ENV
git diff --name-only --diff-filter=ACMRTUXB HEAD~..HEAD | grep -E "\.php$" >> $GITHUB_ENV
git diff --name-only --diff-filter=ACMRTUXB HEAD~..HEAD | grep -E "\.php$" || true >> $GITHUB_ENV
echo 'EOF' >> $GITHUB_ENV
- name: Validate changed files
Expand All @@ -106,5 +110,5 @@ jobs:
grep -v tests/Test/IntegrationCaseFactoryInterface.php |
grep -v tests/Test/InternalIntegrationCaseFactory.php |
grep -v tests/Test/IsIdenticalConstraint.php |
grep -v tests/TestCase.php
grep -v tests/TestCase.php \
&& (echo "UNKNOWN FILES DETECTED" && exit 1) || echo "NO UNKNOWN FILES"
3 changes: 0 additions & 3 deletions dev-tools/composer.json
Expand Up @@ -12,9 +12,6 @@
"phpstan/phpstan": "0.12.18",
"phpstan/phpstan-phpunit": "^0.12"
},
"conflict": {
"hhvm": "*"
},
"config": {
"optimize-autoloader": true,
"sort-packages": true
Expand Down

0 comments on commit eb73769

Please sign in to comment.