Skip to content

Add wp plugin language uninstall --all#128

Merged
danielbachhuber merged 14 commits intomainfrom
fix/122-uninstall-all
Oct 18, 2023
Merged

Add wp plugin language uninstall --all#128
danielbachhuber merged 14 commits intomainfrom
fix/122-uninstall-all

Conversation

@swissspidy
Copy link
Member

Contains follow-up fixes from #124 as well.

Fixes #122.

@swissspidy swissspidy requested a review from a team as a code owner October 18, 2023 14:27
@swissspidy swissspidy added this to the 2.0.16 milestone Oct 18, 2023
Co-authored-by: Daniel Bachhuber <daniel.bachhuber@automattic.com>
@danielbachhuber
Copy link
Member

@swissspidy Does the test failure make sense to you?

@swissspidy
Copy link
Member Author

@swissspidy Does the test failure make sense to you?

For some reason the order of the plugins in the plugin list is different 🤷 Just added some hardening to the test to guard against this.

@danielbachhuber danielbachhuber merged commit 829cdf9 into main Oct 18, 2023
@danielbachhuber danielbachhuber deleted the fix/122-uninstall-all branch October 18, 2023 21:57
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.

Add --all flag to uninstall commands

2 participants