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

implement Guzzle usage. #55

Merged
merged 3 commits into from
Dec 7, 2016

Conversation

dann95
Copy link
Contributor

@dann95 dann95 commented Nov 23, 2016

As commented in #53 , sorry i was too busy but finally do this lil contribution.
i guess that fixes, maybe its good make some Exceptions for case of fail, also to add a try catch for Guzzle.

@dann95
Copy link
Contributor Author

dann95 commented Nov 23, 2016

Uuups seems Guzzle is only available for php > 5.5, is that a problem to this repo?

  • guzzlehttp/guzzle 6.2.2 requires php >=5.5 -> your PHP version (5.4.45) does not satisfy that requirement.
    • guzzlehttp/guzzle 6.2.1 requires php >=5.5 -> your PHP version (5.4.45) does not satisfy that requirement.
    • guzzlehttp/guzzle 6.2.0 requires php >=5.5.0 -> your PHP version (5.4.45) does not satisfy that requirement.

@dann95 dann95 mentioned this pull request Dec 7, 2016
@anhskohbo anhskohbo merged commit 5831a8a into anhskohbo:master Dec 7, 2016
@anhskohbo
Copy link
Owner

Thanks so much! I'm so sorry for delay.

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.

None yet

2 participants