-
Notifications
You must be signed in to change notification settings - Fork 3.5k
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
Drop support for PHP 7.4 #3634
Comments
And it will be. |
I see. |
And that case, can you please re-add support for PHP 8.0 (see #3684)? It used to support PHP 8.0 in previous releases just fine, e.g. 1.25.2 . Or update the docs. Anyway, thanks for all the hard work that has gone into the PhpSpreadsheet project, it is very useful. |
Php 8.0 is fully supported. For the issue to which you link, it seems you are running Php8.0 with Zipstream v3, which does not support 8.0; however, PhpSpreadsheet supports Zipstream V2 and V3. See #3615 (comment) for a means to clear your problem. |
Fix PHPOffice#3634. Its time has come. Based on our experience dropping 7.3, I anticipate problems with Github Actions.
This is according to our formal, published, policy to only support EOL PHP after 6 months. See https://phpspreadsheet.readthedocs.io/en/latest/#php-version-support Also share the exact same dev deps across all PHP version for GitHub Actions so runs are faster, much most importantly they are stable and predictable. And we decide manually when we want to migrate to PHPUnit 10. Fixes #3634 Closes #3710
This is according to our formal, published, policy to only support EOL PHP after 6 months. See https://phpspreadsheet.readthedocs.io/en/latest/#php-version-support Also share the exact same dev deps across all PHP version for GitHub Actions so runs are faster, much most importantly they are stable and predictable. And we decide manually when we want to migrate to PHPUnit 10. Fixes #3634 Closes #3710
This is:
As stated in the README, this project supports PHP 7.4 until 28th June 2023.
So, the minimum supported PHP version should be raised to 8.0
The text was updated successfully, but these errors were encountered: