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

Use guzzle 5 instead of guzzle 6? #10

Closed
markodobric opened this Issue Jan 21, 2016 · 3 comments

Comments

Projects
None yet
3 participants
@markodobric

markodobric commented Jan 21, 2016

Hi Steven,

I'm using your great trello wrapper in my project, but I have terrible dependencies hell. There a lot of other stuff which already use guzzle 5 and I have package conflict because your package use guzzle 6.

Is there a possibility that your package uses guzzle 5 instead of guzzle 6?

Thanks.

@stevenmaguire

This comment has been minimized.

Owner

stevenmaguire commented Jan 21, 2016

That will not be possible. There was a significant change in guzzle from 5 to 6, hence the major version bump, and the changes in 6 drive a lot of functionality in this project. Among the changes are support for PSR 7 Http Messages, some valuable standards that may improve your own project.

@ilijastuden

This comment has been minimized.

ilijastuden commented Jan 26, 2016

How about if we refactor Http layer to use HttpPlug?

https://packagist.org/packages/php-http/httplug

This way Guzzle6 will be an suggested dependncy, and end user would be able to add any HTTP communication library as transport.

@ilijastuden

This comment has been minimized.

ilijastuden commented Jan 26, 2016

Of course, we would make a Pull Request, no need for you to develop it. Just checking to see if you will consider merging it if we developed it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment