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

Update expression for dependencies of PHPUnit #362

Merged
merged 2 commits into from May 14, 2021

Conversation

@yukihiko-shinoda
Copy link
Contributor

@yukihiko-shinoda yukihiko-shinoda commented Aug 24, 2020

The latest version of WordPress supports PHPUnit 7.x.

@see https://core.trac.wordpress.org/ticket/50482#comment:8
@see https://make.wordpress.org/core/handbook/testing/automated-testing/phpunit/#setup

I checked that my maintaining CI for WordPress plugin works well with WordPress latest and PHPUnit 7.x.
Build #27 - yukihiko-shinoda/staticpress2019 - Travis CI

* [The latest version of WordPress supports PHPUnit 7.x](https://make.wordpress.org/core/handbook/testing/automated-testing/phpunit/#setup)
* The legacy version of WordPress supported PHPUnit 5.x only when running PHP 7
* The legacy version of WordPress required PHPUnit 4.8 when running PHP 5
Copy link
Member

@schlessera schlessera Aug 26, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It would actually be helpful if the precise WordPress versions were mentioned, like "WordPress 5.5+ (is this correct) supports PHPUnit 7.x".

Loading

Copy link
Contributor Author

@yukihiko-shinoda yukihiko-shinoda Aug 26, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I found Supported Version Chart:
PHPUnit Compatibility and WordPress Versions – Make WordPress Core

Is it better to replace all with link to this page?

Loading

Copy link
Member

@schlessera schlessera May 14, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, that way we don;t have to maintain the exact requirements in here.

Loading

Copy link
Contributor Author

@yukihiko-shinoda yukihiko-shinoda May 14, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Updated.

Loading

Replace explanation of support PHPUnit version
with link to Supported Version Chart in WordPress core handbook.
@schlessera schlessera added this to the 2.5.0 milestone May 14, 2021
@schlessera
Copy link
Member

@schlessera schlessera commented May 14, 2021

Thanks for the PR, @yukihiko-shinoda !

Loading

@schlessera schlessera merged commit 17387e4 into wp-cli:master May 14, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants