-
Notifications
You must be signed in to change notification settings - Fork 86
Add support for GitHub release installation #421
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
Add support for GitHub release installation #421
Conversation
src/WP_CLI/CommandWithUpgrade.php
Outdated
| if ( wp_remote_retrieve_response_code( $response ) === \WP_Http::FORBIDDEN ) { | ||
| return new \WP_Error( | ||
| \WP_Http::FORBIDDEN, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks like WP_Http::FORBIDDEN doesn't exist yet on WordPress 3.7, see https://github.com/wp-cli/extension-command/actions/runs/8851555744/job/24308388075?pr=421#step:11:67
I suppose we can just hardcode 403 here, maybe adding an explanatory comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed up in fc16a6a
danielbachhuber
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, @tfirdaus !
--------- Co-authored-by: Bogdan Ungureanu <bogdanungureanu21@gmail.com> Co-authored-by: Daniel Bachhuber <daniel.bachhuber@automattic.com>

Formed in 2009, the Archive Team (not to be confused with the archive.org Archive-It Team) is a rogue archivist collective dedicated to saving copies of rapidly dying or deleted websites for the sake of history and digital heritage. The group is 100% composed of volunteers and interested parties, and has expanded into a large amount of related projects for saving online and digital history.

Related to #91 and resuming #386.