-
-
Notifications
You must be signed in to change notification settings - Fork 42
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
Drop php5 #225
Drop php5 #225
Conversation
Might be a good occasion to add php |
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.
I'm @greg0ire and I approve this PR
@soullivaneuh Your job :) |
Even I quite personally agree with that, PHP 5.6 still have security update. In my very personal opinion, I would also like to remove the support of this kind of version. But Sonata packages are also used by @rande company and it's a quite important change that make lot of debates on each open-source project. I would like the "Go" of @rande before doing this. |
Hello, New majors are there to move forward, so I will be happy to drop php 5 on the next major. |
Thanks @rande ! @soullivaneuh , please merge and apply, this blocks #216 |
Thank you @core23 |
Based on the twitter vote, we could drop PHP5 completly for the next majors.