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 Mar 21, 2024
1 parent 3cf70c6 commit 1b3fed2
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,15 @@ 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
- mysql:8
- mariadb:10
- mariadb:11
php:
- 8.3
- 8.2
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 1b3fed2

Please sign in to comment.