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 requirements to PHP 8.0 #665

Merged
merged 1 commit into from
Jan 13, 2023
Merged

🔧 Bump requirements to PHP 8.0 #665

merged 1 commit into from
Jan 13, 2023

Conversation

retlehs
Copy link
Member

@retlehs retlehs commented Jan 2, 2023

https://www.php.net/supported-versions.php
https://endoflife.date/php

I'd prefer to bump this to PHP 8.1, considering active support for PHP 8.0 ended on 2022-11-26, but WordPress & WP-CLI still don't fully support it at this time

Ref https://roots.io/wordpress-php-8-1-support-not-found/
Ref https://core.trac.wordpress.org/ticket/54504 (coming in WordPress 6.2)
Ref wp-cli/wp-cli#5623

@retlehs retlehs merged commit d28b231 into master Jan 13, 2023
@retlehs retlehs deleted the drop-php-7.4 branch January 13, 2023 19:35
@drzraf
Copy link

drzraf commented Apr 26, 2023

I believe it's an error.
Those relying on PHP 7.x (and whatever the reasons are, it's their problem) should be given the choice. If a dependency actually needs 8.x, it's fine but please don't arbitrarily increase the version number. WP still supports a broad range of PHP versions (including 7.x) and that's fine.

@retlehs
Copy link
Member Author

retlehs commented Apr 26, 2023

It’s actually not fine that WordPress supports versions of PHP that aren’t actively supported.

Bedrock is just a boilerplate. Feel free to change the value to whatever you’d like for your own projects — no one is forcing you to use a certain version of PHP. We’ll be bumping this to PHP 8.1 now that WordPress 6.2 is out.

@roots roots locked as resolved and limited conversation to collaborators Apr 26, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants