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

Allow guzzle 7 #308

Merged
merged 4 commits into from
Jul 14, 2020
Merged

Allow guzzle 7 #308

merged 4 commits into from
Jul 14, 2020

Conversation

mihaileu
Copy link
Contributor

@mihaileu mihaileu commented Jul 6, 2020

Q A
Bug fix no
New feature yes
BC breaks no
Deprecations no
Tests pass yes
Fixed tickets [comma-separated list of tickets fixed by the PR, if any]
License MIT

@gregurco gregurco self-requested a review July 6, 2020 09:01
@gregurco
Copy link
Member

gregurco commented Jul 6, 2020

Hello @mihaileu . Did you test bundle with v7? Does it work well?

@mihaileu
Copy link
Contributor Author

mihaileu commented Jul 6, 2020

Hello @mihaileu . Did you test bundle with v7? Does it work well?

I didn't tested it yet. I'll need to run some test but it might take some time. Also I'm not sure I'll cover all the features.

@gregurco
Copy link
Member

I tested and it's ok 👍 Thanks!

@gregurco gregurco merged commit 84b0f13 into 8p:master Jul 14, 2020
@tjveldhuizen
Copy link
Contributor

At which term can we expect a release containing this change? I'ld like to use Guzzle 7, however this bundle is blocking it.

@gregurco
Copy link
Member

@tjveldhuizen thanks for reminder. Just released all changes: https://github.com/8p/EightPointsGuzzleBundle/releases/tag/v8.2.0

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.

3 participants