-
-
Notifications
You must be signed in to change notification settings - Fork 825
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
Bump Minimum Required PHP to be 5.6 #13309
Bump Minimum Required PHP to be 5.6 #13309
Conversation
(Standard links)
|
@seamuslee001 we should do this for sure. I think we should possibly also increase the const MIN_RECOMMENDED_PHP_VER = '7.0'; variable - since this is the last month of php support for it |
@eileenmcnaughton are we prepared to recommend anything higher than 7.1 at this stage? |
And in fact const RECOMMENDED_PHP_VER = '7.1'; - we should be recommending php 7.2 now IMHO |
ah snap - YES I think so - there is the smtp password version but I think we just live with the advice that people re-save their passwords when updating to php 7.2 |
done that now @eileenmcnaughton |
OK - looks good to me! |
Jenkins re test this please |
@seamuslee001 this is the wrong branch I just realised - 5.9 will be the last php 5.5 supporting version |
@eileenmcnaughton switched to master |
Merging as per tag |
Overview
This bumps the minimum required PHP version to be 5.6 as per https://civicrm.org/blog/eileen/end-of-life-plans-for-5x-php-versions-planning-for-70-eol
Before
Previously minimum php version was 5.5
After
Minimum php version is 5.6
ping @totten @eileenmcnaughton