Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

GH Actions: expand the test matrix #316

Merged
merged 2 commits into from
May 4, 2022
Merged

Conversation

jrfnl
Copy link
Member

@jrfnl jrfnl commented May 4, 2022

GH Actions: run at least one test build with iconv disabled

GH Actions: expand the test matrix

This commit expands the test matrix to test against more PHP/PHPCS combinations, thus extending cross-version support safeguarding.

The tests will now be run for most PHP versions against a wider range of PHPCS versions.

  • Builds already run in quick test/code coverage are removed (excluded) as there is no need to run those twice.
  • Builds against PHP 8.0+ are explicitly defined via includes as most older PHPCS versions cannot be supported due to a fatal error (which was fixed in PHPCS 3.5.0).

Includes updating the documentation for the matrix.

Refs:

jrfnl added 2 commits May 4, 2022 16:43
This commit expands the test matrix to test against more PHP/PHPCS combinations, thus extending cross-version support safeguarding.

The tests will now be run for most PHP versions against a wider range of PHPCS versions.
* Builds already run in quick test/code coverage are removed (`exclude`d) as there is no need to run those twice.
* Builds against PHP 8.0+ are explicitly defined via `include`s as most older PHPCS versions cannot be supported due to a fatal error (which was fixed in PHPCS 3.5.0).

Includes updating the documentation for the matrix.

Refs:
* https://docs.github.com/en/actions/using-jobs/using-a-matrix-for-your-jobs
@jrfnl jrfnl added this to the 1.0.0-alpha4 milestone May 4, 2022
@jrfnl jrfnl merged commit 9ecd297 into develop May 4, 2022
@jrfnl jrfnl deleted the feature/ghactions-expand-matrix branch May 4, 2022 15:22
@jrfnl jrfnl mentioned this pull request Oct 15, 2022
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant