Skip to content

Commit aa6178d

Browse files
authored
Update PHPCS and PHP versions in unit tests (#3)
This pull request updates the PHP and PHPCS version matrix in the GitHub Actions workflow for unit tests. The main goal is to ensure compatibility with the latest PHP and PHPCS versions and to improve test coverage for development versions. **Updates to test matrix:** - **Expanded PHP version support:** * Added PHP 8.5 to the test matrix to ensure future compatibility. * Included a job to test PHP 8.5 with PHPCS `4.x-dev` for early detection of compatibility issues. - **Expanded PHPCS version support:** * Added `3.x-dev` to the PHPCS versions that are tested. - **Fix branch naming** * Updated the job that previously used `dev-master` for PHPCS to use `3.x-dev`, aligning with current PHPCS branch naming. This fixes job failure that used `dev-master` when the `master` branch was renamed to `3.x`. - **Workflow trigger improvements:** * Modified the `pull_request` trigger in `.github/workflows/unit-tests.yml` to ignore changes to unit test workflow files themselves, preventing redundant workflow runs when only those files are changed.
2 parents 365c88a + e8157be commit aa6178d

1 file changed

Lines changed: 9 additions & 3 deletions

File tree

.github/workflows/unit-tests.yml

Lines changed: 9 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,11 @@ on:
88
paths-ignore:
99
- "**.md"
1010
- "**.xml"
11+
1112
pull_request:
13+
# Don't run on pull requests that only change unit test workflow files.
14+
paths-ignore:
15+
- "**unit-tests.yml"
1216
# Allow manually triggering the workflow.
1317
workflow_dispatch:
1418

@@ -63,8 +67,8 @@ jobs:
6367
# @link https://docs.github.com/en/free-pro-team@latest/actions/reference/workflow-syntax-for-github-actions#jobsjob_idstrategymatrix
6468
#
6569
# The matrix is set up so as not to duplicate the builds which are run for code coverage.
66-
php: ["7.2", "7.3", "7.4", "8.0", "8.1", "8.2", "8.3", "8.4"]
67-
phpcs_version: ["lowest", "stable", "4.x-dev"]
70+
php: ["7.2", "7.3", "7.4", "8.0", "8.1", "8.2", "8.3", "8.4", "8.5"]
71+
phpcs_version: ["lowest", "stable", "4.x-dev", "3.x-dev"]
6872

6973
exclude:
7074
- php: "8.3"
@@ -77,13 +81,15 @@ jobs:
7781

7882
# Test against dev versions of all dependencies with select PHP versions for early detection of issues.
7983
- php: "7.2"
80-
phpcs_version: "dev-master"
84+
phpcs_version: "3.x-dev"
8185
- php: "7.2"
8286
phpcs_version: "4.x-dev"
8387
- php: "7.4"
8488
phpcs_version: "4.x-dev"
8589
- php: "8.2"
8690
phpcs_version: "4.x-dev"
91+
- php: "8.5"
92+
phpcs_version: "4.x-dev"
8793

8894
name: "Test: PHP ${{ matrix.php }} - PHPCS ${{ matrix.phpcs_version }}"
8995

0 commit comments

Comments
 (0)