Skip to content

Commit

Permalink
Merge pull request #109 from wp-cli/fix-workflow-step-id
Browse files Browse the repository at this point in the history
  • Loading branch information
swissspidy authored Oct 1, 2024
2 parents 0ca5afc + bed9720 commit b82beed
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions .github/workflows/reusable-testing.yml
Original file line number Diff line number Diff line change
Expand Up @@ -245,7 +245,7 @@ jobs:
run: echo "VERSION=$(vendor/bin/phpunit --version | grep --only-matching --max-count=1 --extended-regexp '\b[0-9]+\.[0-9]+')" >> $GITHUB_OUTPUT

- name: Determine PHPUnit flavour
id: phpunit_version
id: phpunit_flavour
run: |
if [ startsWith( steps.phpunit_version.outputs.VERSION, '10.' ) }}" == "true" ]; then
echo 'FLAVOUR=gte10' >> $GITHUB_OUTPUT
Expand All @@ -256,7 +256,7 @@ jobs:
# PHPUnit 10 may fail a test run when the "old" configuration format is used.
# Luckily, there is a build-in migration tool since PHPUnit 9.3.
- name: Migrate PHPUnit configuration for PHPUnit 10+
if: ${{ steps.phpunit_version.outputs.FLAVOUR == 'gte10' }}
if: ${{ steps.phpunit_flavour.outputs.FLAVOUR == 'gte10' }}
continue-on-error: true
run: composer phpunit -- --migrate-configuration

Expand Down

0 comments on commit b82beed

Please sign in to comment.