-
Notifications
You must be signed in to change notification settings - Fork 71
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
Update to guzzle 6 #29
base: master
Are you sure you want to change the base?
Conversation
…nto FU-3661/update-to-guzzle6 Conflicts: .scrutinizer.yml
[FU-3661] Update to guzzle6
👍 The build fails because Guzzle requires PHP 5.5 as minimum versions and the only test that fails on Travis is the one on PHP 5.4... All the others pass... |
Fix scrutinizer/travis issues
@lemasson-h Would you please rebase from master and squash your commits? It is now quite hard to see the changes you made. |
@@ -32,36 +24,33 @@ First make sure you have all the dependencies in place by running `composer inst | |||
use GuzzleHttp\Client; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
use GuzzleHttp\HandlerStack;
Here an approach to try to be as close as possible to the previous version adapted for guzzle6.
Would you be happy with that?