Skip to content

Commit 2a20bf3

Browse files
authored
Merge pull request #279 from PaulRotmann/PHP8.4
Run tests on PHP 8.4 and update test environment + PCOV to avoid segfault with Xdebug 3.4.2
2 parents f2fb5a2 + 0b2a3c3 commit 2a20bf3

File tree

1 file changed

+8
-6
lines changed

1 file changed

+8
-6
lines changed

.github/workflows/ci.yml

Lines changed: 8 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -7,10 +7,11 @@ on:
77
jobs:
88
PHPUnit:
99
name: PHPUnit (PHP ${{ matrix.php }})
10-
runs-on: ubuntu-22.04
10+
runs-on: ubuntu-24.04
1111
strategy:
1212
matrix:
1313
php:
14+
- 8.4
1415
- 8.3
1516
- 8.2
1617
- 8.1
@@ -24,10 +25,10 @@ jobs:
2425
- uses: shivammathur/setup-php@v2
2526
with:
2627
php-version: ${{ matrix.php }}
27-
coverage: xdebug
28+
coverage: ${{ matrix.php < 8.0 && 'xdebug' || 'pcov' }}
2829
ini-file: development
2930
ini-values: disable_functions='' # do not disable PCNTL functions on PHP < 8.1
30-
extensions: sockets, pcntl, event, ev
31+
extensions: sockets, pcntl, event, ${{ matrix.php < 8.0 && 'ev-1.1.5' || 'ev' }}
3132
env:
3233
fail-fast: true # fail step if any extension can not be installed
3334
- run: composer install
@@ -38,7 +39,7 @@ jobs:
3839

3940
PHPUnit-Unstable:
4041
name: PHPUnit (Unstable PHP ${{ matrix.php }})
41-
runs-on: ubuntu-22.04
42+
runs-on: ubuntu-24.04
4243
continue-on-error: true
4344
strategy:
4445
matrix:
@@ -56,7 +57,7 @@ jobs:
5657
- uses: shivammathur/setup-php@v2
5758
with:
5859
php-version: ${{ matrix.php }}
59-
coverage: xdebug
60+
coverage: ${{ matrix.php < 8.0 && 'xdebug' || 'pcov' }}
6061
ini-file: development
6162
extensions: sockets, pcntl
6263
- name: Install ext-uv
@@ -77,6 +78,7 @@ jobs:
7778
strategy:
7879
matrix:
7980
php:
81+
- 8.4
8082
- 8.3
8183
- 8.2
8284
- 8.1
@@ -90,7 +92,7 @@ jobs:
9092
- uses: shivammathur/setup-php@v2
9193
with:
9294
php-version: ${{ matrix.php }}
93-
coverage: xdebug
95+
coverage: ${{ matrix.php < 8.0 && 'xdebug' || 'pcov' }}
9496
ini-file: development
9597
extensions: sockets,event # future: add uv-beta (installs, but can not load)
9698
- run: composer install

0 commit comments

Comments
 (0)