Skip to content

Updated to Guzzle5#79

Merged
jessespears merged 5 commits intomailgun:masterfrom
Nyholm:guzzle
May 6, 2015
Merged

Updated to Guzzle5#79
jessespears merged 5 commits intomailgun:masterfrom
Nyholm:guzzle

Conversation

@Nyholm
Copy link
Collaborator

@Nyholm Nyholm commented Apr 18, 2015

Guzzle5 and some other neat tricks makes the RestClient a lot cleaner.

This PR does not change the API nor does it add any features. I think this is a necessary update because it reduces the number of dependencies in my application. I don't want dependencies on both Guzzle3 and Guzzle5.

Also, with the new Guzzle we do no longer suffer from the bug temporarily fixed here

@Nyholm
Copy link
Collaborator Author

Nyholm commented Apr 18, 2015

This PR resolve issue #73

@Nyholm
Copy link
Collaborator Author

Nyholm commented May 5, 2015

Rebased. Ping @jessespears.

jessespears pushed a commit that referenced this pull request May 6, 2015
@jessespears jessespears merged commit 9b153c1 into mailgun:master May 6, 2015
@jessespears
Copy link

Thanks for the contribution and sorry about the long wait! Please send an email to me at jesse@mailgun.com with your info so we can send you a Mailgun t-shirt.

@Nyholm Nyholm deleted the guzzle branch May 6, 2015 05:23
@Nyholm
Copy link
Collaborator Author

Nyholm commented May 6, 2015

Thank you for merging this. I'm happy to contribute to make this project better.

PS, don't forget to close #73

@Nyholm
Copy link
Collaborator Author

Nyholm commented May 6, 2015

When can we expect version 1.8.0 to be tagged?

@jessespears
Copy link

Today =)

On Wed, May 6, 2015 at 7:09 AM, Tobias Nyholm notifications@github.com
wrote:

When can we expect version 1.8.0 to be tagged?


Reply to this email directly or view it on GitHub
#79 (comment).

@Nyholm
Copy link
Collaborator Author

Nyholm commented May 6, 2015

You are awesome @jessespears!

@jessespears
Copy link

Thanks for waiting, version 1.8 is tagged! You are awesome @Nyholm!

@afrowuk
Copy link

afrowuk commented Jul 9, 2015

Will there be an update to Guzzle6?

@auro1
Copy link

auro1 commented Aug 27, 2015

#86

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.

4 participants