Tag for stable for Silex 2 about predisserviceprovider HOT 6 CLOSED

Tagged with no changes for the namespace of this provider.

from predisserviceprovider.

Same here :)

from predisserviceprovider.

Sorry for being so inactive on this project, I'll try to make up for it soon 😄 As for a new branch and release targeting Silex 2... yes it's going to happen, I just have a doubt if the Predis\Silex namespace should remain as is or if we could rename it as Predis\Pimple, this service provider doesn't really depend on Silex anymore after all. Any thoughts on this?

from predisserviceprovider.

No worries. I just figured the master branch is working fine for me so why not tag it :) I'm fine with changing the namespace to I'm fine withPimple` too, makes sense.

from predisserviceprovider.

I've no strong opinion on the move to another namespace.
For example the Silex providers have been made independent of the Silex Application and rely on Pimple only, yet they are still in the Silex namespace (for example https://github.com/silexphp/Silex-Providers/blob/master/AssetServiceProvider.php).

from predisserviceprovider.

Thank you @nrk !

from predisserviceprovider.