Combine test run without pcntl into one job

revoltphp · Apr 2, 2023 · 873d784 · 873d784
1 parent e638121
commit 873d784
Showing 1 changed file with 6 additions and 11 deletions.
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -13,17 +13,9 @@ jobs:
         include:
           - operating-system: 'ubuntu-latest'
             php-version: '8.1'
-            composer-flags: '--ignore-platform-req=php'
-            phpunit-flags: ''
-            phpunit-description: ''
-          - operating-system: 'ubuntu-latest'
-            php-version: '8.1'
-            composer-flags: '--ignore-platform-req=php'
-            phpunit-flags: '-d disable_functions=pcntl_signal_dispatch'
-            phpunit-description: 'without pcntl'
+            composer-flags: '--ignore-platform-req=php+'
 
     name: PHP ${{ matrix.php-version }} ${{ matrix.job-description }}
-
     runs-on: ${{ matrix.operating-system }}
 
     steps:
@@ -72,8 +64,11 @@ jobs:
             php -v
             composer info -D
 
-      - name: Run tests ${{ matrix.phpunit-description }}
-        run: php ${{ matrix.phpunit-flags }} vendor/bin/phpunit
+      - name: Run tests
+        run: php vendor/bin/phpunit
+
+      - name: Run tests without pcntl
+        run: php -d disable_functions=pcntl_signal_dispatch vendor/bin/phpunit
 
       - name: Run Psalm
         run: vendor/bin/psalm.phar --show-info=true