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

feat: switch to FrankenPHP #460

Merged
merged 1 commit into from
Sep 20, 2023
Merged

feat: switch to FrankenPHP #460

merged 1 commit into from
Sep 20, 2023

Conversation

dunglas
Copy link
Owner

@dunglas dunglas commented Sep 13, 2023

Use FrankenPHP 🎉

@bzoks
Copy link

bzoks commented Sep 14, 2023

Wouldn't it be "better" to add this as an alternative/parallel option (for example in a separate Dockerfile or at least a separate stage), besides php-fpm, especially because of this part in the readme:

FrankenPHP is very experimental, don't use it in production yet

Starting to use dockerized PHP can be a large step for newcomers, but in parallel going away from well-known php-fpm... could be a showstopper for some (too adventurous)

@dunglas dunglas force-pushed the feat/frankenphp branch 2 times, most recently from fab0faa to dfa0929 Compare September 14, 2023 12:46
@dunglas
Copy link
Owner Author

dunglas commented Sep 14, 2023

@bzoks I plan to create an fpm branch to still support FPM for a while. However, in the long term, the goal is to switch to FrankenPHP because it has been designed especially for this kind of use case. It is slimmer, easier to use, and faster than FPM.

Dockerfile Outdated Show resolved Hide resolved
@dunglas dunglas merged commit 88f5c19 into main Sep 20, 2023
2 checks passed
@dunglas dunglas deleted the feat/frankenphp branch September 20, 2023 14:53
AchillesKal pushed a commit to AchillesKal/php-docker that referenced this pull request Nov 28, 2023
LucasHospice pushed a commit to LucasHospice/symfony-docker that referenced this pull request Feb 21, 2024
VitaliyMinenko pushed a commit to VitaliyMinenko/notofication-service that referenced this pull request Sep 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants