Skip to content

Commit

Permalink
Add rdbms matrix for wider compatibility testing
Browse files Browse the repository at this point in the history
This change set introduces a `rdbms` matrix with the purpose of increasing insurance this package works with up to the latest MySQL/MariaDB versions.

Note that due to MySQL's versioning there are no `v6` and `v7`.

Refs: friends-of-reactphp#194
  • Loading branch information
WyriHaximus committed Apr 23, 2024
1 parent 3cf70c6 commit 249c5fe
Showing 1 changed file with 7 additions and 2 deletions.
9 changes: 7 additions & 2 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -6,10 +6,12 @@ on:

jobs:
PHPUnit:
name: PHPUnit (PHP ${{ matrix.php }})
name: PHPUnit (PHP ${{ matrix.php }}; RDBMS ${{ matrix.rdbms }})
runs-on: ubuntu-22.04
strategy:
matrix:
rdbms:
- mysql:5
php:
- 8.3
- 8.2
Expand All @@ -23,6 +25,9 @@ jobs:
- 5.6
- 5.5
- 5.4
include:
- php: 8.3
rdbms: mariadb:10
steps:
- uses: actions/checkout@v4
- uses: shivammathur/setup-php@v2
Expand All @@ -31,7 +36,7 @@ jobs:
coverage: xdebug
ini-file: development
- run: composer install
- run: docker run -d --name mysql --net=host -e MYSQL_RANDOM_ROOT_PASSWORD=yes -e MYSQL_DATABASE=test -e MYSQL_USER=test -e MYSQL_PASSWORD=test mysql:5
- run: docker run -d --name mysql --net=host -e MYSQL_RANDOM_ROOT_PASSWORD=yes -e MYSQL_DATABASE=test -e MYSQL_USER=test -e MYSQL_PASSWORD=test ${{ matrix.rdbms }}
- run: bash tests/wait-for-mysql.sh
- run: vendor/bin/phpunit --coverage-text
if: ${{ matrix.php >= 7.3 }}
Expand Down

0 comments on commit 249c5fe

Please sign in to comment.