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

Bump required PHP version to 7.2 #1408

Open
borekb opened this issue Mar 26, 2019 · 1 comment
Open

Bump required PHP version to 7.2 #1408

borekb opened this issue Mar 26, 2019 · 1 comment
Assignees
Labels
scope: system requirements Things like PHP versions, Git versions, supported OS / web servers, etc.
Milestone

Comments

@borekb
Copy link
Member

borekb commented Mar 26, 2019

When working on #1389, I realized that we only run tests on PHP 7.2 which made me think whether we shouldn't bump the required version anyway. Some points around this:

  • PHP 5.6, VersionPress' current minimum required version (Bump required PHP version to 5.6+ #726, docs), has been unsupported for almost two years now:
    Screenshot 2019-03-26 at 11 00 44
    PHP 7.2 is the lowest fully supported version as of today.
  • The performance of PHP 7 is much better.
  • We could use language features like type hints.
  • PHPUnit has a policy of only running on the currently supported PHP versions, also tools like PHPStan require recent PHP releases to run.

A counter-argument is that PHP 5.6 is still massively popular. Incidentally, WordPress just announced that WP 5.2 will bump the minimum required version to PHP 5.6 and a chart in that blog post shows that PHP 5.6 is used on 32% of WordPress sites – in fact, it's the most popular version out there, followed by PHP 7.0 with 18%. PHP 7.2 is third with 14%:

Screenshot 2019-03-26 at 11 29 19

Still, in my eyes, it's more important to make VersionPress work reliably than to worry about legacy PHP versions. In practice, VersionPress can be run only on servers that the user has a good control of (Git is required, etc.) so I think requiring supported PHP version is not that big of an ask.

@borekb borekb added the scope: core Core VersionPress functionality like tracking actions, creating Git commits, etc. label Mar 26, 2019
@borekb borekb added this to the 4.0 milestone Mar 26, 2019
@borekb borekb self-assigned this Mar 26, 2019
@borekb borekb added scope: system requirements Things like PHP versions, Git versions, supported OS / web servers, etc. and removed scope: core Core VersionPress functionality like tracking actions, creating Git commits, etc. labels Mar 27, 2019
@borekb
Copy link
Member Author

borekb commented Apr 1, 2019

This, and I don't think it's an April Fools' 😄

Screenshot 2019-04-01 at 17 13 08

Source.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
scope: system requirements Things like PHP versions, Git versions, supported OS / web servers, etc.
Projects
4.0-beta2 → 4.0-beta3
  
Awaiting triage
Development

No branches or pull requests

1 participant