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

Psalm integration #17

Closed boesing closed 1 year ago

boesing commented 4 years ago

Feature Request

Q A
QA yes

Summary

As decided during the Technical-Steering-Committee Meeting on August 3rd, 2020, Laminas wants to implement vimeo/psalm in all packages.

Implementing psalm is quite easy.

Required

Optional
tux-rampage commented 4 years ago

@boesing composer require vimeo/psalm would add it as a runtime dependency. Shouldn't this be composer require --dev vimeo/psalm In general I'd even prefer composer global require vimeo/psalm locally and in CI to not mess with the dependency tree. Unlike PHPstan, psalm is not distributed with isolated dependencies.

michalbundyra commented 4 years ago

@tux-rampage yes, it should be --dev