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

Avoid fatal errors when pulling in the bundle with Requests v2 via Composer #552

Merged
merged 8 commits into from
Jun 3, 2023

Conversation

schlessera
Copy link
Member

Trying to trigger the issue described in https://github.com/wp-cli/wp-cli#5795

@schlessera schlessera requested a review from a team as a code owner June 2, 2023 09:45
@schlessera
Copy link
Member Author

The changes in wp-cli/wp-cli#5796 do actually solve the issue now. The tests up here are still broken, as it is not easy to get the version in the pull request be pulled in via Composer (as it will then break after merge. So I'm merging this now with the failure, expecting the tests to pass once this is merged into main.

@schlessera schlessera added this to the 2.8.1 milestone Jun 3, 2023
@schlessera
Copy link
Member Author

From my initial testing now, it looks like this also allows you to use WP-CLI to run core updates across this v1/v2 boundary in both directions without any issues.

@schlessera schlessera merged commit bca70d7 into main Jun 3, 2023
@schlessera schlessera deleted the fix/wp-cli-bundle-composer-fatal branch June 3, 2023 06:45
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.

1 participant