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.

Sign up for GitHub

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

Jump to bottom

Drop nyholm/psr7 dependency #85

Closed
JaZo opened this issue Nov 24, 2020 · 4 comments
Closed

Drop nyholm/psr7 dependency #85

JaZo opened this issue Nov 24, 2020 · 4 comments
Milestone
2.0

Comments

@JaZo
Copy link
Member

JaZo commented Nov 24, 2020

Detailed description

Drop nyholm/psr7 dependency.

Context

When guzzle/psr7#327 is closed and guzzle/psr7 has a new major we can drop this dependency and leave it to the consumer.
@JaZo JaZo added this to the 2.0 milestone Nov 24, 2020
@bbrala bbrala mentioned this issue Nov 24, 2020
Closed
@bbrala
Copy link
Member

bbrala commented Jun 18, 2021

Well, RC1 has been out for a while, but we are waiting for stable i guess?

@JaZo
Copy link
Member Author

JaZo commented Jun 22, 2021 •
edited
Loading

Guzzle added support for guzzle/psr7 v2 in 7.3.0, so if your minimum stability is set to RC you'll get v2. I think it's safe to drop nyholm/psr7.

@bbrala
Copy link
Member

bbrala commented Jun 22, 2021

Yeah i agree :)

@JaZo
Copy link
Member Author

JaZo commented Jul 1, 2021

https://github.com/guzzle/psr7/releases/tag/2.0.0 🎉

JaZo added a commit that referenced this issue Jul 1, 2021
@JaZo
Drop nyholm/psr7 dependency
3d21982 
Guzzle now includes PSR-17 factories.

Closes: #85
@JaZo JaZo closed this as completed Jul 1, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
2.0
Development

No branches or pull requests
2 participants
@bbrala @JaZo