search

mcrumm / phlack

Slack Integrations for PHP
MIT License
52 stars 8 forks source link

"Package guzzle/guzzle is abandoned" when installing #35

Open maaarghk opened 8 years ago

maaarghk commented 8 years ago

HI,

When installing this package, composer throws the following warning:

Package guzzle/guzzle is abandoned, you should avoid using it. Use guzzlehttp/guzzle instead.

I'm not sure how backwards compatible this new package is but it might be worth looking into doing an upgrade.

mcrumm commented 8 years ago

Sorry for the super late response!

Phlack is based on Guzzle3, and while there really isn't a lot going on in the service descriptions, Guzzle6 (latest) doesn't currently work with guzzle-services.

The service description portion of Phlack could be refactored to simply use guzzlehttp/command, which does work with Guzzle6.

If you're interested in submitting a PR for that change, I would happily consider updating the Guzzle dependency.

I don't have a lot of time to devote to Phlack right now, so unless someone contributes this change, I likely won't get around to updating anytime soon.

mcrumm commented 8 years ago

@rdohms Out of curiosity, would an upgrade to Guzzle6 create any major issues for sloth?