-
-
Notifications
You must be signed in to change notification settings - Fork 6
Add PHP FPM #211
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
base: master
Are you sure you want to change the base?
Add PHP FPM #211
Conversation
# Conflicts: # Dockerfile-fpm-alpine # Dockerfile-fpm-debian
|
Waiting for docker-library/official-images#13602 to be merged. |
|
@frankvanhest I think you swapped some commit around 😅 |
What do you mean? |
|
These are the changes for including FPM tags, not changes in the excludes file as I suggested here: #210 (comment) |
|
I see what you mean I think. I’ll try to look into it as soon as possible. It is a bit busy at home at the moment with family and so. |
|
Take your time, family and so is always more important :) |
|
@frankvanhest FYI since |
|
@WyriHaximus, I jus saw your message. It is not yet ready to be merged. It still needs testing. So don't wait for this to continue rolling out PHP 8.2. |
|
@frankvanhest Cool, so this will become the FPM pr? Then I'll update the title. And I'm currently fighting xdebug for 8.2 😂 . |
|
@WyriHaximus, I completely forgot about this PR. In June 2023 I got a burnout unfortunately. Since a while I've been well again after some time. |
|
@frankvanhest I don't have a personal use for it. But please continue to work on it, I will merge it in and provide it. Also been through a burnout myself since then. Glad to see you back, and please take care in the future. It wasn't a good experience IMHO. P.S. I'm currently travelling for work, when I get back I'll do a review of the changes. |
No description provided.