-
Notifications
You must be signed in to change notification settings - Fork 111
Added PHP 7.3 support - updated Travis CI configuration #223
base: develop
Are you sure you want to change the base?
Added PHP 7.3 support - updated Travis CI configuration #223
Conversation
@@ -18,7 +18,7 @@ All notable changes to this project will be documented in this file, in reverse | |||
|
|||
### Removed | |||
|
|||
- Nothing. | |||
- [#223](https://github.com/zendframework/zend-mail/pull/223) removes support for zend-stdlib v2 releases. |
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.
why?
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.
v2 is no longer supported: https://framework.zend.com/long-term-support
and it's not fully compatible with PHP 7.3. No point to keep it here.
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.
include info in commit message and/or pr body. not just dumbly remove, then the maintainers don't have to ask either, but can already decide.
This repository has been closed and moved to laminas/laminas-mail; a new issue has been opened at laminas/laminas-mail#20. |
This repository has been moved to laminas/laminas-mail. If you feel that this patch is still relevant, please re-open against that repository, and reference this issue. To re-open, we suggest the following workflow:
|
No description provided.