leanphp / behat-code-coverage

Generate Code Coverage reports for Behat tests
BSD 2-Clause "Simplified" License
50 stars 23 forks source link

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

Closed thijswitteveen closed 7 years ago

ruudy-es commented 7 years ago

+1

litzinger commented 7 years ago

Why is Guzzle 4 still being used on this project?

dgtlmoon commented 7 years ago

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

afurculita commented 7 years ago

@ek9 any news on this?

ek9 commented 7 years ago

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:

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.