search

jelovac / bitly4laravel

Provides a Laravel package to communicate with Bit.ly API
35 stars 18 forks source link

Upgrade to Guzzle v6 #21

Closed PhiloNL closed 7 years ago

jelovac commented 9 years ago

HI, I won't merge this for now because a lot of sites still use php 5.4 and the guzzle v6 requires 5.5.

azorgh commented 9 years ago

Why not authorize Guzzle V5 AND V6 ? Or maybe create another branch ?

PhiloNL commented 8 years ago

Any update on this? Currently other packages that use V6 are forced to use V5 which is causing some issues. Any chance you could create a separate branch and release as 4.0?

azorgh commented 8 years ago

You can use my own version : https://github.com/MyoStudio/bitly4laravel/tree/fixGuzzleVersionTo6.0

PhiloNL commented 8 years ago

@PallMallShow Thanks

mbarwick83 commented 8 years ago

+1 to create a separate branch for Guzzle 6. Please @jelovac 👯

mbarwick83 commented 8 years ago

@pallmallshow how did your fork to work? I'm running Laravel 5.2.

"repositories": [
    {
        "type": "vcs",
        "url": "https://github.com/MyoStudio/bitly4laravel"
    }
],
"require": {
    ...
    "jelovac/bitly4laravel": "dev-master",
}