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

Require PHP 7.2+ #1988

Closed
franzliedke opened this issue Feb 7, 2020 · 2 comments · Fixed by #2014
Closed

Require PHP 7.2+ #1988

franzliedke opened this issue Feb 7, 2020 · 2 comments · Fixed by #2014

Comments

@franzliedke
Copy link
Contributor

Let's give everyone a two months heads-up, and then dump PHP 7.1 (does not even receive security fixes anymore) for beta.13.

Related:

@franzliedke franzliedke added this to the 0.1.0-beta.13 milestone Feb 7, 2020
@tankerkiller125
Copy link
Contributor

This seems like a very reasonable time frame, especially since every major PHP project I know of has already been updated to support 7.2 and 7.3, and all Linux distros that I know of ship with 7.2 as the default or newer. Plus if we manage to update our illuminate components in beta.13 7.1 would be unsupported anyways.

@tankerkiller125
Copy link
Contributor

This would enable further Laravel upgrades. Any LTS we can fix our eyes on?

I just now noticed this line here, Laravel is at 6.0 LTS which will receive security updates until September of 2022. As a note Laravel switched to Semantic versioning, as such the upgrade process going forward will be from version 6.0 -> X.0 instead of the old 5.0 -> 5.X

The proper composer version string for the LTS of laravel components is ^6.0 the following is the comparison between our version (5.7.x) and the latest LTS release (6.0) and does not include all the latest patches (which are non-breaking) laravel/framework@v5.7.28...v6.0.0

franzliedke added a commit that referenced this issue Mar 17, 2020
Now that we require PHP 7.2, this ensures we get the latest updates and
fixes as well.

Refs #1988.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants