Attempted fix for PHP 8.2 tests

tvdijen · tvdijen · commit fc71570c4b99 · 2023-01-05T17:01:29.000+01:00
diff --git a/.github/workflows/php.yml b/.github/workflows/php.yml
@@ -14,6 +14,8 @@ jobs:
   basic-tests:
     name: Syntax and unit tests, PHP ${{ matrix.php-versions }}, ${{ matrix.operating-system }}
     runs-on: ${{ matrix.operating-system }}
+    env:
+      NO_COVERAGE: '--no-coverage'
     strategy:
       fail-fast: false
       matrix:
@@ -68,17 +70,17 @@ jobs:
         run: bash vendor/bin/check-syntax-php.sh
 
       - name: Decide whether to run code coverage or not
-        if: ${{ matrix.php-versions != '8.0' || matrix.operating-system != 'ubuntu-latest' }}
+        if: ${{ matrix.php-versions == '8.1' || matrix.operating-system == 'ubuntu-latest' }}
         run: |
-          echo "NO_COVERAGE=--no-coverage" >> $GITHUB_ENV
+          echo "NO_COVERAGE=" >> $GITHUB_ENV
 
       - name: Run unit tests
         run: |
           echo $NO_COVERAGE
           ./vendor/bin/phpunit $NO_COVERAGE
 
       - name: Save coverage data
-        if: ${{ matrix.php-versions == '8.0' && matrix.operating-system == 'ubuntu-latest' }}
+        if: ${{ matrix.php-versions == '8.1' && matrix.operating-system == 'ubuntu-latest' }}
         uses: actions/upload-artifact@v3
         with:
             name: build-data
