Skip to content

Allow installation of psr/http-message v2.0 #1313

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

Closed
wants to merge 1 commit into from

Conversation

gjuric
Copy link
Contributor

@gjuric gjuric commented Apr 26, 2023

Related to elastic/elastic-transport-php#17

I've signed the CLA in the meantime.

@elasticmachine
Copy link
Collaborator

Since this is a community submitted pull request, a Jenkins build has not been kicked off automatically. Can an Elastic organization member please verify the contents of this patch and then kick off a build manually?

@gjuric
Copy link
Contributor Author

gjuric commented May 23, 2023

The tests are failing because php-http/client-common in v2.7 dropped the dependency on php-http/message-factory.

See getsentry/sentry-php#1533 for a similar case.

@ezimuel
Copy link
Contributor

ezimuel commented May 23, 2023

I fixed the issue and merged in main manually. Thanks!

@ezimuel ezimuel closed this May 23, 2023
@ezimuel
Copy link
Contributor

ezimuel commented Jun 5, 2023

@gjuric I had to remove the tag v8.8.1 because it has been released before the official schedule. The tag v8.8.1 will be re-created soon in a few days. Sorry for any inconvenience.

@gjuric
Copy link
Contributor Author

gjuric commented Jun 6, 2023

Thanks for the heads up.

@ezimuel
Copy link
Contributor

ezimuel commented Jun 9, 2023

Released v8.8.1 with #1313

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants