-
-
Notifications
You must be signed in to change notification settings - Fork 29
Conversation
8b7906b
to
98264f0
Compare
792e754
to
8b6e631
Compare
Signed-off-by: Stefan Van de Kaa <[email protected]>
8b6e631
to
2641c59
Compare
Sorry for the many updates. was my first PR on the Laminas project. Know now how to do it proper for the next time :) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM - minor changes needed, but after those, it can be merged
Signed-off-by: Stefan Van de Kaa <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 👍
LGTM! What is potentially still a merge blocker (not to be done here - need to ask in the contributors slack channel) is whether we need to add the release automation toolchain here 🤔 |
Decided to release manually: component is deprecated and shouldn't be maintained, but indeed you already put in the effort to get it moving forward, so let's roll with a manual release :-) |
Released under https://github.com/laminas/laminas-mvc-console/releases/tag/1.3.0 - thanks @stefanvandekaa! |
closes: #8
Signed-off-by: Stefan Van de Kaa [email protected]