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

Discussion the drop support of PHP v7.3 #179

Closed
94noni opened this issue Jun 5, 2021 · 3 comments · Fixed by #284
Closed

Discussion the drop support of PHP v7.3 #179

94noni opened this issue Jun 5, 2021 · 3 comments · Fixed by #284

Comments

@94noni
Copy link
Contributor

94noni commented Jun 5, 2021

Hi,

I began using MeiliSearch recently, and with PHP for a personal project

Regarding:

I am opening this issue to discuss the minimum version of this package, I think requiring PHP v7.4 could improve the codebase for the better, and will be very happy to contribute to :)

Feel free to close, related to #136 and #165

Cheers!

@alallema
Copy link
Contributor

alallema commented Jun 7, 2021

Hi @94noni,
Thank you for raising this issue. Until now we keep the versions until they are no longer supported (end of 2021 for 7.3), we don't want to rush people.
But it's a very relevant question and I'd be glad to have some opinions from the community about it. So don't hesitate to comment, I let this issue open.

@codedge
Copy link
Contributor

codedge commented Jun 21, 2021

There is now a nice live PHP version stats view: https://packagist.org/php-statistics

So currently

  • 20% still run on 7.3.
  • 65% run on 7.4 & 8.0.

we don't want to rush people.

That is a good one 😄 .. but I get your point.

I think 6th Dec 2021 when 7.3 security support ends there can be a hard cut dropping 7.3. People then had 1 year of no rushing since the active support stopped for 7.3 😉

And the good thing is that people who still wants to use this lib with php 7.3 they still can. It is just that they probably won't benefit from future additions to this lib. And this is one major point where I can understand to adopt new versions earliert and drop old ones. There is no downside for the people. And if there is a major bug upcoming you can still think about backporting the fix.

@esmultivac
Copy link

I think the current policy is a good one. No doubt that the code can be smaller and more polished dropping support for not so old versions, but it can be hard for some projects to move on to the newer ones. We're currently working on upgrading from 7.2 to 8.0 and it's a ton of work. It's a work that must be done eventually, of course, and the end of support is a very good milestone to force it.

@mmachatschek mmachatschek mentioned this issue Jan 19, 2022
3 tasks
bors bot added a commit that referenced this issue Jan 20, 2022
284: Drop PHP 7.3 support r=alallema a=mmachatschek

# Pull Request

## What does this PR do?
Fixes #179

This sets the minimum PHP-version to 7.4 which still receives security fixes. PHP 7.3 has [reached its EOL](https://www.php.net/supported-versions.php) in december.

## PR checklist
Please check if your PR fulfills the following requirements:
- [x] Does this PR fix an existing issue?
- [x] Have you read the contributing guidelines?
- [x] Have you made sure that the title is accurate and descriptive of the changes?


Co-authored-by: Markus Machatschek <markus.machatschek@hey.com>
Co-authored-by: Amélie <alallema@users.noreply.github.com>
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

Successfully merging a pull request may close this issue.

4 participants