Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added laminas/laminas-servicemanager as non-dev dependency, required at runtime #145

Merged
merged 1 commit into from
Feb 21, 2022

Conversation

gsteel
Copy link
Member

@gsteel gsteel commented Feb 21, 2022

Q A
Bugfix yes
BC Break no
QA yes

Description

Psr\Container and Interop\Container are too…

This is something I thought would be good to get into 2.20 before it gets released - In 3.0, if we introduce a config provider, this will mean that we'll be able to import the psalm types for factory configuration etc. I think that ^3.6 is the feasible minimum for service menager for this, currently though, I'd find it unlikely that installing view on PHP 7.4 gets you anything less that 3.10 of service manager.

Psr\Container and Interop\Container are too.

Signed-off-by: George Steel <george@net-glue.co.uk>
@Ocramius Ocramius added this to the 2.20.0 milestone Feb 21, 2022
composer.json Show resolved Hide resolved
@Ocramius Ocramius self-assigned this Feb 21, 2022
Copy link
Member

@Ocramius Ocramius left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @gsteel!

@Ocramius Ocramius merged commit 689753f into laminas:2.20.x Feb 21, 2022
@Ocramius Ocramius changed the title Laminas ServiceManager is a hard dependency Added laminas/laminas-servicemanager as non-dev dependency, required at runtime Feb 21, 2022
@gsteel gsteel deleted the require-container branch February 21, 2022 18:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants