Skip to content

Conversation

@Xerkus
Copy link
Member

@Xerkus Xerkus commented Mar 27, 2020

Q A
Documentation yes
Bugfix no
BC Break no
New Feature no
RFC no
QA no

Description

This PR updates stratigility to 2.2.2, updates tests, deprecates direct usage of double-pass and callable middlewares to mirror stratigility forward-compat changes. No functionality change is brought by this update.

Xerkus added a commit to Xerkus/laminas-mvc-middleware that referenced this pull request Mar 27, 2020
Signed-off-by: Aleksei Khudiakov <aleksey@xerkus.pro>
Xerkus added a commit to Xerkus/laminas-mvc-middleware that referenced this pull request Mar 27, 2020
Signed-off-by: Aleksei Khudiakov <aleksey@xerkus.pro>
@Xerkus Xerkus force-pushed the feature/update-stratigility branch from b726809 to 84c77cd Compare March 27, 2020 23:57
@Xerkus Xerkus added this to the 1.1.0 milestone Mar 28, 2020
Copy link
Member

@michalbundyra michalbundyra left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Xerkus in general LGTM, just one small comment, please.

Copy link
Member

@weierophinney weierophinney left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just a couple documentation suggestions; otherwise, looks fine.

Xerkus added 4 commits March 31, 2020 08:22
Signed-off-by: Aleksei Khudiakov <aleksey@xerkus.pro>
Signed-off-by: Aleksei Khudiakov <aleksey@xerkus.pro>
Signed-off-by: Aleksei Khudiakov <aleksey@xerkus.pro>
Signed-off-by: Aleksei Khudiakov <aleksey@xerkus.pro>
@Xerkus Xerkus force-pushed the feature/update-stratigility branch from 84c77cd to 2f518d6 Compare March 30, 2020 22:36
@Xerkus
Copy link
Member Author

Xerkus commented Mar 30, 2020

@weierophinney @michalbundyra thank you

@Xerkus Xerkus merged commit 476c292 into laminas:develop Mar 31, 2020
@Xerkus Xerkus deleted the feature/update-stratigility branch March 31, 2020 02:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants