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

Update Guzzle to 5.0 #1

Merged
merged 1 commit into from
Dec 10, 2014
Merged

Update Guzzle to 5.0 #1

merged 1 commit into from
Dec 10, 2014

Conversation

dunglas
Copy link
Contributor

@dunglas dunglas commented Dec 1, 2014

To allow using the SDK on project using the latest stable Guzzle version.

@gierschv
Copy link

gierschv commented Dec 5, 2014

Hi,

Thank you for your contribution, however could you sign your commit (git commit --amend -s --no-edit && git push -f), as required in the CONTRIBUTING guidelines.

Thank you by advance,
Vincent

Signed-off-by: Kévin Dunglas <dunglas@gmail.com>
@dunglas
Copy link
Contributor Author

dunglas commented Dec 8, 2014

@gierschv:

  • Added backward compatibility with Guzzle 4
  • Signed the commit

@gierschv
Copy link

gierschv commented Dec 9, 2014

Thanks! LGTM (ping @VincentCasse)

@VincentCasse
Copy link
Member

@dunglas Thanks for you pull request
@gierschv thanks for your review :)

Merged

VincentCasse added a commit that referenced this pull request Dec 10, 2014
Add Guzzle to 5.0 compatibility
@VincentCasse VincentCasse merged commit 9308018 into ovh:master Dec 10, 2014
rbeuque74 pushed a commit that referenced this pull request Feb 3, 2016
(fix): add formatting to endpoint information
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.

3 participants