search

laminas / laminas-di

Automated dependency injection for PSR-11 containers
https://docs.laminas.dev/laminas-di/
BSD 3-Clause "New" or "Revised" License
36 stars 20 forks source link

Drop container-interop #68

Closed remicollet closed 2 years ago

remicollet commented 2 years ago

which is replaced by recent servicemanager

Signed-off-by: Remi Collet remi@famillecollet.com

Q A
Documentation no
Bugfix no
BC Break no
New Feature no
RFC no
QA yes
Ocramius commented 2 years ago

@remicollet thanks, will update composer.lock before merging :+1:

remicollet commented 2 years ago

@Ocramius notice that there is a set of laminas-* which can be improved the same way

Ocramius commented 2 years ago

@remicollet yeah, the idea there would be that @renovatebot would take those up, but I haven't had the time to set it up organization-wide yet.

Lemme do it here first :-)

Ocramius commented 2 years ago

Had to expand the patch a bit: PHP_CodeSniffer was right in finding pending Interop\Container\ references that had to be removed.

Did my best to keep everything BC compliant, by adding a conflict with earlier laminas/laminas-servicemanager releases (since that component is not a strict requirement for this library to be used).