-
Notifications
You must be signed in to change notification settings - Fork 4
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
Make Guzzle package optional #40
Conversation
e495471
to
fb1cb49
Compare
Please merge #41 first. |
I see there is still guzzle |
Some updates from v5 to v6, we could take a look at here As I knew, we are using |
Yes you should update to Guzzle 6.0 |
2d15751
to
ca4c62a
Compare
I did, thanks @saimaz. |
How's about this, guys? |
ca4c62a
to
84dad47
Compare
Can you rebase it to one commit? |
84dad47
to
a2f0c6b
Compare
Yes, I did |
Closes #34.