8000 Merge pull request #31 from clue-labs/php8.2 · clue/reactphp-ndjson@c37981a · GitHub
[go: up one dir, main page]
Skip to content

Commit c37981a

Browse files
SimonFringsSimonFrings
authored
Merge pull request #31 from clue-labs/php8.2
Test on PHP 8.2 and update test environment
2 parents c127b2b + 770c9a2 commit c37981a

File tree

3 files changed

+25
-16
lines changed

3 files changed

+25
-16
lines changed

‎.github/workflows/ci.yml

Lines changed: 17 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -6,10 +6,12 @@ on:
66

77
jobs:
88
PHPUnit:
9-
runs-on: ubuntu-latest
9+
name: PHPUnit (PHP ${{ matrix.php }})
10+
runs-on: ubuntu-22.04
1011
strategy:
1112
matrix:
1213
php:
14+
- 8.2
1315
- 8.1
1416
- 8.0
1517
- 7.4
@@ -22,11 +24,11 @@ jobs:
2224
- 5.4
2325
- 5.3
2426
steps:
25-
- uses: actions/checkout@v2
26-
- name: Setup PHP
27-
uses: shivammathur/setup-php@v2
27+
- uses: actions/checkout@v3
28+
- uses: shivammathur/setup-php@v2
2829
with:
2930
php-version: ${{ matrix.php }}
31+
coverage: xdebug
3032
- run: composer install
3133
- run: vendor/bin/phpunit --coverage-text
3234
if: ${{ matrix.php >= 7.3 }}
@@ -35,12 +37,16 @@ jobs:
3537

3638
PHPUnit-hhvm:
3739
name: PHPUnit (HHVM)
38-
runs-on: ubuntu-18.04
40+
runs-on: ubuntu-22.04
41+
continue-on-error: true
3942
steps:
40-
- uses: actions/checkout@v2
41-
- uses: azjezz/setup-hhvm@v1
43+
- uses: actions/checkout@v3
44+
- run: cp "$(which composer)" composer.phar && ./composer.phar self-update --2.2 # downgrade Composer for HHVM
45+
- name: Run hhvm composer.phar install
46+
uses: docker://hhvm/hhvm:3.30-lts-latest
4247
with:
43-
version: lts-3.30
44-
- run: composer self-update --2.2 # downgrade Composer for HHVM
45-
- run: hhvm $(which composer) install
46-
- run: hhvm vendor/bin/phpunit
48+
args: hhvm composer.phar install
49+
- name: Run hhvm vendor/bin/phpunit
50+
uses: docker://hhvm/hhvm:3.30-lts-latest
51+
with:
52+
args: hhvm vendor/bin/phpunit

‎composer.json

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -15,8 +15,8 @@
1515
"react/stream": "^1.2"
1616
},
1717
"require-dev": {
18-
"react/event-loop": "^1.2",
19-
"phpunit/phpunit": "^9.3 || ^5.7 || ^4.8.35"
18+
"phpunit/phpunit": "^9.5 || ^5.7 || ^4.8.35",
19+
"react/event-loop": "^1.2"
2020
},
2121
"autoload": {
2222
"psr-4": { "Clue\\React\\NDJson\\": "src/" }

‎phpunit.xml.dist

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,8 @@
11
<?xml version="1.0" encoding="UTF-8"?>
22

3-
<!-- PHPUnit configuration file with new format for PHPUnit 9.3+ -->
4-
<phpunit xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
5-
xsi:noNamespaceSchemaLocation="https://schema.phpunit.de/9.3/phpunit.xsd"
3+
<!-- PHPUnit configuration file with new format for PHPUnit 9.5+ -->
4+
<phpunit xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
5+
xsi:noNamespaceSchemaLocation="https://schema.phpunit.de/9.5/phpunit.xsd"
66
bootstrap="vendor/autoload.php"
77
cacheResult="false"
88
colors="true"
@@ -17,4 +17,7 @@
1717
<directory>./src/</directory>
1818
</include>
1919
</coverage>
20+
<php>
21+
<ini name="error_reporting" value="-1" />
22+
</php>
2023
</phpunit>

0 commit comments

Comments
 (0)
0