Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support laminas-code:v4 #23

Merged
merged 5 commits into from
Jan 8, 2021
Merged

Support laminas-code:v4 #23

merged 5 commits into from
Jan 8, 2021

Conversation

Slamdunk
Copy link
Contributor

@Slamdunk Slamdunk commented Jan 8, 2021

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

Needed for a full PHP8 support.

Supersede #21

Signed-off-by: Filippo Tessarotto <zoeslam@gmail.com>
Signed-off-by: Filippo Tessarotto <zoeslam@gmail.com>
Signed-off-by: Matthew Weier O'Phinney <matthew@weierophinney.net>
CHANGELOG.md Outdated
@@ -22,6 +22,8 @@ All notable changes to this project will be documented in this file, in reverse

### Removed

- [#23](https://github.com/laminas/laminas-server/pull/23) removes support for laminas-code releases prior to 4.0.0.
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Signed-off-by: Matthew Weier O'Phinney <matthew@weierophinney.net>
Remove unused import statement

Signed-off-by: Matthew Weier O'Phinney <matthew@weierophinney.net>
@weierophinney weierophinney added this to the 3.0.0 milestone Jan 8, 2021
@weierophinney weierophinney merged commit abe5480 into laminas:3.0.x Jan 8, 2021
@Slamdunk Slamdunk deleted the laminas_code_4 branch January 28, 2021 12:35
@Slamdunk
Copy link
Contributor Author

@weierophinney I'd like to land this into my production. It is not my intention to rush you through the upcoming 3.0.0, if it's not the right time yet: in that case, would you accept a similar PR against 2.9.x branch?

@froschdesign
Copy link
Member

@Slamdunk
Please try to ask in the "contributers" channel of the chat or create a new issue report. Thanks!

@Slamdunk Slamdunk mentioned this pull request Mar 24, 2021
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.

None yet

3 participants