refactor: update dependencies and remove deprecated properties#136
refactor: update dependencies and remove deprecated properties#136gsteel merged 3 commits intolaminas:5.0.xfrom
Conversation
| "laminas/laminas-servicemanager": "^4.0", | ||
| "laminas/laminas-servicemanager": "^4.4.0", | ||
| "laminas/laminas-stdlib": "^3.20", | ||
| "psr/container": "^1.1", |
There was a problem hiding this comment.
Hi @gsteel,
Is it necessary for laminas/laminas-hydrator to depend on laminas/laminas-modulemanager?
It seems that laminas/laminas-modulemanager only provides default configuration for Laminas MVC applications.

Since Laminas MVC is being retired (announcement), would it make sense to decouple this dependency?
There was a problem hiding this comment.
The laminas-modulemanager support can be removed. See: laminas/laminas-validator#384
4fe1fc5 to
2401318
Compare
gsteel
left a comment
There was a problem hiding this comment.
Can you please drop module-manager stuff in a separate PR?
This makes it easier to formulate change-logs/migration docs etc.
Cheers 👍
| "laminas/laminas-modulemanager": "^2.17", | ||
| "laminas/laminas-serializer": "^3.0", | ||
| "laminas/laminas-servicemanager": "^4.0", | ||
| "laminas/laminas-servicemanager": "^4.4.0", |
There was a problem hiding this comment.
Making Laminas Service Manager a hard dep is the right thing to do - I hate soft deps, but, Hydrator ships a standalone container specifically to avoid installation of servicemanager, so I'm not sure this is ideal. WDYT @froschdesign?
Signed-off-by: mmalac <mmalac@sygic.com>
…alysis Signed-off-by: mmalac <mmalac@sygic.com>
…itattributes Signed-off-by: mmalac <mmalac@sygic.com>
ce59a03 to
ebd4eb7
Compare

Description
This PR adds
composer-dependency-analyserto improve static analysis of unused or missing dependencies in the project.I used
maglnet/composer-require-checker, but I found thatshipmonk-rnd/composer-dependency-analyseris more effective:During this process, I resolved the issues flagged by the tool and also updated some Composer dependencies to ensure compatibility and clean analysis results.
Added a conditional polyfill for the
#[AllowDynamicProperties]attribute, which is only available in PHP 8.2+