-
Notifications
You must be signed in to change notification settings - Fork 9.4k
min php version fix #19455
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
min php version fix #19455
Conversation
as written here: https://devdocs.magento.com/guides/v2.3/install-gde/system-requirements-tech.html the min php version is 7.1.3 but the check in this file was wrong (so also the error messages)
Hi @fballiano. Thank you for your contribution
For more details, please, review the Magento Contributor Assistant documentation |
Hi @orlangur, thank you for the review. |
@fballiano thank you for contributing. Please accept Community Contributors team invitation here to gain extended permissions for this repository. |
@magento-engcom-team please include this in 2.3.0 release if possible. Also, would be nice to add this to some release activity checklist or a "bump minimum required version" doc as in 2.2 bootstrap became out of sync as well: https://github.com/magento/magento2/blob/2.2-develop/composer.json#L11 |
Ok, nevermind, 2.3.0 is already released 🤣 |
Hi @fballiano. Thank you for your contribution. |
Just a small update for the minimum PHP version and the link to the documentation
Description (*)
as written here: https://devdocs.magento.com/guides/v2.3/install-gde/system-requirements-tech.html the min php version is 7.1.3 but the check in this file was wrong (so also the error messages)
Fixed Issues (if relevant)
Manual testing scenarios (*)