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

Tests: allow for PHPUnit 8/9 #809

Merged
merged 1 commit into from
Feb 11, 2024
Merged

Conversation

jrfnl
Copy link
Collaborator

@jrfnl jrfnl commented Feb 11, 2024

Follow up on #805

PHP_CodeSniffer 3.8.0 now allows for running the tests, which are based on the PHPCS native test suite, with PHPUnit 8 and 9.

This commit updates the package to take advantage of that.

Includes:

  • Widening the PHPUnit version requirements.
  • Adding the PHPUnit 8+ cache file to .gitignore.
  • Updating the PHPUnit configuration to make sure that PHP deprecations will always show, even when on a high PHPUnit 9.6 version.
  • Updating the info in CONTRIBUTING.
  • Simplifications to the quicktest and test workflows. Also, the code coverage "high" run can now be run against PHP 8.3.
  • Running PHPStan against PHP latest (couldn't previously be done due to the old PHPUnit version).
  • Updating the unit-tests script to remove the PHP version based toggle, which is no longer needed.

Ref:

PHP_CodeSniffer 3.8.0 now allows for running the tests, which are based on the PHPCS native test suite, with PHPUnit 8 and 9.

This commit updates the package to take advantage of that.

Includes:
* Widening the PHPUnit version requirements.
* Adding the PHPUnit 8+ cache file to `.gitignore`.
* Updating the PHPUnit configuration to make sure that PHP deprecations will always show, even when on a high PHPUnit 9.6 version.
* Updating the info in `CONTRIBUTING`.
* Simplifications to the `quicktest` and `test` workflows.
    Also, the code coverage "high" run can now be run against PHP 8.3.
* Running PHPStan against PHP `latest` (couldn't previously be done due to the old PHPUnit version).
* Updating the `unit-tests` script to remove the PHP version based toggle, which is no longer needed.

Ref:
* PHPCSStandards/PHP_CodeSniffer 59
@jrfnl jrfnl added this to the 3.x milestone Feb 11, 2024
@jrfnl jrfnl requested a review from a team as a code owner February 11, 2024 01:40
@GaryJones GaryJones merged commit 5ee0bce into develop Feb 11, 2024
40 checks passed
@GaryJones GaryJones deleted the feature/phpunit-cross-version branch February 11, 2024 12:20
jrfnl added a commit that referenced this pull request Feb 11, 2024
@jrfnl jrfnl mentioned this pull request Feb 11, 2024
rebeccahum pushed a commit that referenced this pull request May 2, 2024
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.

2 participants