Skip to content

Failed plugin updates now throw errors#143

Merged
schlessera merged 6 commits intowp-cli:masterfrom
austinginder:fix-90-plugin-update-reporting
Dec 12, 2018
Merged

Failed plugin updates now throw errors#143
schlessera merged 6 commits intowp-cli:masterfrom
austinginder:fix-90-plugin-update-reporting

Conversation

@austinginder
Copy link
Contributor

Resolve issue #90. This checks for a null response from bulk_upgrade and assumes a plugin update did not occur. That may include more errors than intended as plugin update errors are not included in response. Refer to https://developer.wordpress.org/reference/classes/plugin_upgrader/.

@schlessera schlessera added the command:plugin-update Related to 'plugin update' command label Dec 9, 2018
@schlessera schlessera added this to the 2.0.2 milestone Dec 9, 2018
austinginder and others added 3 commits December 9, 2018 15:54
Co-Authored-By: austinginder <austinginder@gmail.com>
…der/extension-command into fix-90-plugin-update-reporting
@schlessera schlessera merged commit 5c7fbd9 into wp-cli:master Dec 12, 2018
@schlessera
Copy link
Member

Thanks for the PR, @austinginder !

@schlessera schlessera changed the title 🐛 FIX: Failed plugin updates not throwing errors Failed plugin updates now throw errors Dec 12, 2018
@austinginder
Copy link
Contributor Author

@schlessera Sure thing and thanks for helping me work through the Behat test.

@schlessera schlessera modified the milestones: 2.0.2, 2.0.3 Dec 17, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

command:plugin-update Related to 'plugin update' command

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants