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

Remove support for outdated PHP versions #72

Open
pixelbrackets opened this issue May 25, 2022 · 2 comments
Open

Remove support for outdated PHP versions #72

pixelbrackets opened this issue May 25, 2022 · 2 comments

Comments

@pixelbrackets
Copy link
Contributor

pixelbrackets commented May 25, 2022

The current project supports outdated PHP versions as low as 5.2, which is EOL for more than 11 years now.

Issue #69 showed that is not even possible to build the required test images for these versions anymore. Tests for these versions have been dropped in PR #71.

@sgiehl wrote:

Feel free to remove PHP 5.6 here as well. We also should update the readme that we require at least PHP 7. Older versions of PHP are meanwhile EOL.

My proposal: Streamline this with the current version of matomo/matomo and drop support for all versions < 7.2. They are all EOL.

It is reasonable to keep support for PHP 7.2 as it is part of running distros like Ubuntu 18.04, although it should be recommended everywhere to update to 8.0.

Proposed steps:

@pixelbrackets pixelbrackets changed the title Remove support for ancient PHP versions Remove support for outdated PHP versions May 25, 2022
@sgiehl
Copy link
Member

sgiehl commented May 25, 2022

@pixelbrackets I would fully agree with your proposed steps.
@tsteur @mattab Do you see any problems in dropping support for older PHP versions here?

@tsteur
Copy link
Member

tsteur commented May 27, 2022

Since it's a small project and people might include it in all kind of setups it would be great to ideally still support PHP 7.0 and newer from my perspective and it be fine to drop support for PHP 5.X as long as we release a new major version then.

Would it cause any issues to also support PHP 7.0 and 7.1 re the tests?

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

No branches or pull requests

3 participants