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

Need support for laminas/laminas-servicemanager v4 #206

Open
hartois opened this issue Sep 11, 2023 · 5 comments · May be fixed by #267
Open

Need support for laminas/laminas-servicemanager v4 #206

hartois opened this issue Sep 11, 2023 · 5 comments · May be fixed by #267
Assignees
Milestone

Comments

@hartois
Copy link

hartois commented Sep 11, 2023

laminas/laminas-validator cannot be installed with laminas/laminas-servicemanager v4, that support psr/container v2.
Need support.

@hartois hartois added the Bug Something isn't working label Sep 11, 2023
@froschdesign
Copy link
Member

So far there is no version 4 of laminas-servicemanager. If there is a new version, then this package is also updated.

@hartois
Copy link
Author

hartois commented Sep 11, 2023

So far there is no version 4 of laminas-servicemanager. If there is a new version, then this package is also updated.

RC v4 of laminas-servicemanager exists.
Now laminas-validator (and laminas-mail as example that have laminas-validator dependency) cannot be installed in projects, that need psr/container v2.

Ok, we need wait for release of laminas-servicemanager?

@boesing boesing added Feature Request and removed Bug Something isn't working labels Sep 11, 2023
@boesing
Copy link
Member

boesing commented Sep 11, 2023

Ok, we need wait for release of laminas-servicemanager?

No, you have to wait until compatibility is added to this package.
ServiceManager v4 requires major versions of almost all components, not just validator.

There is no priority on when which package will get updated first. If you want to help, feel free to provide a patch. Happy to merge stuff even tho just adding support for SM v4 will not be the only thing we have to add for the next major.

@Zombaya
Copy link
Contributor

Zombaya commented Jun 25, 2024

Looks like PR #267 will solve this issue.

@gsteel
Copy link
Member

gsteel commented Jun 25, 2024

Yes @Zombaya - it's a breaking change so it will be part of v3.0

@gsteel gsteel linked a pull request Jun 25, 2024 that will close this issue
4 tasks
@gsteel gsteel self-assigned this Jun 25, 2024
@gsteel gsteel added this to the 3.0.0 milestone Jun 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants