10up / wp-local-docker-v2

ARCHIVED: A simple Docker based development environment for WordPress.
MIT License
484 stars 82 forks source link

PHP 8.2 support #332

Closed justinmaurerdotdev closed 1 year ago

justinmaurerdotdev commented 1 year ago

Is your enhancement related to a problem? Please describe.

In the words of stitcher.io:

PHP 8.0 went into security fixes only mode at the end of last year, and I hope to see its usage decline a lot more this year. Keep in mind that PHP 8.0 will reach end of life on November 26, 2023. So it's crucial that projects start preparing to upgrade in the coming months.

PHP 7.4 reached end of life last year, so at the same time it's worrying that more than 25% of projects are still using it! Let's hope to see this number decline soon.

Our team is really enjoying the 10updocker workflow. We want to jump from 7.4 to 8.2 (because of the features available in 8.2), but we don't want to have to give up the 10updocker setup! There's an old, approved PR for adding 8.1, but it's never been merged. And now it's old enough that 8.2 should really be the default.

The author of that ticket says

While it's easy to switch to 8.1. by editing docker-composer.yml and restarting, this PR would make things a little quicker.

I attempted it, and things started breaking. If no 8.1/8.2 support will be added to the 10updocker setup, do you think you could add some documentation for how to upgrade the PHP image?

I guess, in general, it would also be nice to have some clarification about the status of this project. I know it has been a year since any changes were made to the master branch.

Thanks in advance for any kind of official response here.

Designs

No response

Describe alternatives you've considered

No response

Code of Conduct

onwp commented 1 year ago

I agree. PHP 8+ version support should come into this project. I am digging it. If I can manage it, I will send a PR, and you can merge into your local maybe, if the maintainer doesn't merge it. @justinmaurerdotdev

justinmaurerdotdev commented 1 year ago

I decided to try this again, since I noticed that 10up's 8.2 docker image had been updated, and it looks like it...just worked this time.

I just changed the image for the phpfpm container in the docker-compose.yml file to look like this: image: '10up/wp-php-fpm-dev:8.2-ubuntu' and it appears to be functioning just fine.