-
-
Notifications
You must be signed in to change notification settings - Fork 16
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
Add support for PHP 8.3. #81
Conversation
✅ Deploy Preview for lando-php ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
I based this on #51. I am only unsure about the composer version to use. In the image I updated Composer to 2.6.5, but is it preferred to stick to the Composer LTS release 2.2.22? |
Ran a successful test for the basic 8.3 example locally:
|
Redis extension is fixed at 5.3.7 in the Dockerfile. The recent one is https://pecl.php.net/package/redis/6.0.2 Shouln't we use the most recent one instead? Does the 8.3-apache version build for you? I tried to build that Dockerfile locally but when it starts to install imagick it throws this error (with FPM version it builds fine for some reason, maybe I was doing something wrong):
|
I have seen that error earlier today but I can build it now. I have purged my docker image cache while I was working on this, maybe that helps?
I had to stop on this for today. I had some trouble getting the |
Updated to v6.0.2 of the Redis extension in both images. They both can be built successfully. |
@reynoldsalec do you have the privilege to activate the GitHub Actions for this PR? It would be great if we can see the test results. I am getting errors locally when trying to start the
This Due to this issue the tests for these examples also fail locally:
I am using the 3.3-alpha package in the images, and the normal |
Just approved the test runs @pfrenssen, looking at the diffs this looks spot on to me, although it does look like there might be some issue with the imagick build. I think it may just be an incompatibility of imagick with PHP8.3 on Debian/Ubuntu...see Imagick/imagick#630 and mlocati/docker-php-extension-installer#811, looks like they were waiting for the next tagged build to get added, since the master branch of imagick was working for them. If you have interest in becoming a maintainer of the PHP plugin, we should chat, would love to have a PHP wizard (or two) to help keep this plugin up-to-date! |
@reynoldsalec thanks a lot! I'll keep an eye on those two MRs. Thanks for the maintainership offer! I can help out a bit triaging the issue queue, but as I'm just getting my feet wet in the Lando ecosystem (this is my first PR), I don't really have the skills to implement new features. |
For sure @pfrenssen, maybe consider it more a "co-maintainer" type gig...really just having assistance watching the issue queues and doing updates would be a huge help. And we'd be down to do regular meetings with you to go over questions and help you learn deeper Lando concepts. No pressure! :) |
@pfrenssen any support I can give you here? Very much want this for myself this weekend. (Aka happy to debug that failing 8.3 test but don't want to do too much overlap if you are currently debugging it) @reynoldsalec failing test 1 looks like it is about lack of credentials for docker hub
|
@NickDickinsonWilde thanks for the offer! I have invited you to collaborate on the fork. This is not actionable for me right now, so any help is much appreciated. |
Looks like we're good on the test runs now, when we make Lando 3.21 this will be included. Thanks @pfrenssen + @NickDickinsonWilde if you want to upgrade prior to 3.21, you'll need to clone the PHP plugin repo into |
Fixes #77