-
Notifications
You must be signed in to change notification settings - Fork 23
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
WP_VERSION
should default to latest patch if not provided
#51
Comments
@schlessera Can you please assign this task to me? |
As discussed today during hack day, To determine minor and patch versions we will need to rely on https://api.wordpress.org/core/stable-check/1.0/ to get the information on the released WordPress version, but as mentioned in wp-cli/core-command#108 (comment), this endpoint is not cached and result in slow responses or even rate limiting. Since the cc/ @schlessera for visibility. |
Tasks
|
@schlessera Here is the PR to complete task 1 of above tasklist - #199. |
Right now, if you provide the version as
WP_VERSION=5.1
, it will pull in WP version 5.1, even if 5.1.1 is available. This causes issues in tests that rely on the latest WP version, like the checks in thewp-cli/doctor-command
.It would make more sense for the
WP_VERSION
to behave more closely like semantic versioning.So, if 5.1.2 would be the latest available version:
WP_VERSION=5.1
=> "use latest patch verison for the 5.1 minor version" => installs5.1.2
WP_VERSION=5.1.1
=> "use exact patch version 5.1.1" => installs5.1.1
WP_VERSION=5.1.0
=> "use exact patch version 5.1.0" => installs5.1
(as that is the equivalent to5.1.0
.This effectively means that all the
WP_VERSION=x.x
requests will always include all security patches and hotfixes.The text was updated successfully, but these errors were encountered: