Skip to content

Commit

Permalink
Fix tests: old WP versions don't work on PHP 8.1 & 8.2
Browse files Browse the repository at this point in the history
  • Loading branch information
gmazzap committed Mar 1, 2023
1 parent 597d0c2 commit bcaad3e
Showing 1 changed file with 14 additions and 1 deletion.
15 changes: 14 additions & 1 deletion .github/workflows/php-qa.yml
Original file line number Diff line number Diff line change
Expand Up @@ -74,6 +74,17 @@ jobs:
php-ver: [ '7.2', '7.3', '7.4', '8.0', '8.1', '8.2' ]
wp-ver: [ '4.7.*', '5.*', '6.*' ]
dependency-versions: [ 'highest', 'lowest' ]
exclude:
- php-ver: '8.1'
wp-ver: '4.7.*'
- php-ver: '8.1'
wp-ver: '5.*'
dependency-versions: 'lowest'
- php-ver: '8.2'
wp-ver: '4.7.*'
- php-ver: '8.2'
wp-ver: '5.*'
dependency-versions: 'lowest'

steps:

Expand All @@ -100,7 +111,9 @@ jobs:
dependency-versions: ${{ matrix.dependency-versions }}

- name: Run unit tests
run: ./vendor/bin/phpunit ${{ ((env.USE_COVERAGE == 'yes') && '--coverage-html=coverage-report') || '--no-coverage' }}
run: |
./vendor/bin/phpunit --atleast-version 9 && ./vendor/bin/phpunit --migrate-configuration || echo 'Config does not need updates.'
./vendor/bin/phpunit ${{ ((env.USE_COVERAGE == 'yes') && '--coverage-html=coverage-report') || '--no-coverage' }}
- name: Upload coverage report
uses: actions/upload-artifact@v3
Expand Down

0 comments on commit bcaad3e

Please sign in to comment.