search

csarrazi / CsaGuzzleBundle

A bundle integrating Guzzle >=4.0 in Symfony
250 stars 77 forks source link

forcing the middlewares all #213

Closed cordoval closed 6 years ago

cordoval commented 6 years ago 
- Installation request for csa/guzzle-bundle dev-master@dev -> satisfiable by csa/guzzle-bundle[dev-master].

is forcing all middleware packages sad

cordoval commented 6 years ago

perhaps middlewares can be moved to suggest?

cordoval commented 6 years ago

https://github.com/csarrazi/CsaGuzzleBundle/pull/214

csarrazi commented 6 years ago

The separation in multiple packages is only intended for people who are not using the bundle.

The current version of the bundle is plug and play, meaning that all packages should be installed by default. The code of the bundle has not yet been changed yet to check existence or not of the different middleware.

If we move the libraries in suggest, then they won't be automatically installed anymore.

cordoval commented 6 years ago

true