8000 Update Guzzle dependency to require 4.x, 5.x or 6.x by thijswitteveen · Pull Request #13 · leanphp/behat-code-coverage · GitHub
[go: up one dir, main page]

Skip to content

Update Guzzle dependency to require 4.x, 5.x or 6.x#13

Closed
thijswitteveen wants to merge 1 commit intoleanphp:masterfrom
thijswitteveen:master
Closed

Update Guzzle dependency to require 4.x, 5.x or 6.x#13
thijswitteveen wants to merge 1 commit intoleanphp:masterfrom
thijswitteveen:master

Conversation

@thijswitteveen
Copy link

No description provided.

@ruudy-es
Copy link

+1

@litzinger
Copy link

Why is Guzzle 4 still being used on this project?

@dgtlmoon
Copy link

@ek9 any chance of getting this one in? would really help..

@afurculita
Copy link

@ek9 any news on this?

@ek9
Copy link
Member
ek9 commented Oct 17, 2017

Hello,

First, thank you for the PR and contribution. I could not accept this PR as it required updates to RemoteXDebug Driver that depends on guzzle, which were missing. Today I released 2 new versions of this extension:

  • v3.1.0 - uses guzzlehttp/guzzle ~4.0||~5.0. This is legacy support release, mostly for those still stuck with 4.0 or 5.0 for whatever reason. This version will not likely to receive further updates, but can be used for transition period. It is temporarily maintained on v3.1 branch.
  • v3.2.0 - uses guzzlehttp/guzzle ~6.0. This will be maintained and will receive any updates (master branch).

Additionally, as majority seem to not use RemoteXDebug, I am considering making guzzlehttp/guzzle dependency more optional. It would be nice if you could provide any feedback on #15

Closing this as this has been sorted when updating guzzle #16

Thank you for feedback and PR.

@ek9 ek9 closed this Oct 17, 2017
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.

6 participants

0