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

Update Coding Standard to v2 #5

Closed weierophinney closed 4 years ago

weierophinney commented 4 years ago

There are a lot of new conventions that are not covered by the current phpcs ruleset of zend-coding-standard (v1.x).

Some examples are (incomplete list):

A new coding style and tooling is currently work in progress at zendframework/zend-coding-standard#5. When this PR is merged and v2 is released this zend-di should integrate it.

Depending on my availability, I'll perform the tests as suggested in the mentioned PR.


Originally posted by @tux-rampage at https://github.com/zendframework/zend-di/issues/39

gennadigennadigennadi commented 4 years ago

Is there still interested in moving to v2? If it is so, I would like to help. Should I open a PR?

boesing commented 4 years ago

@gennadigennadigennadi Sure!

gennadigennadigennadi commented 4 years ago

@boesing Target Branch is master?

boesing commented 4 years ago

@gennadigennadigennadi 3.2.x as we switched to laminas/automatic-releases 👍