-
Notifications
You must be signed in to change notification settings - Fork 472
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
Symfony 3.4+ #74
Comments
You can see the changes required to support Symfony 4 apps on my fork here: https://github.com/danabrey/docker-symfony-1 |
Hi @danabrey, Edit: I just change PHP version to 7.1 😉 |
Great @maxpou! If it's just the nginx config that needs altering to support a Symfony 4 app, is it worth considering having multiple nginx setups - symfony.dev and symfony4.dev, for example? Just having separate branches would work, though, and is probably clearer. |
First need to change it to something else, for example: Here is the nginx configuration for Symfony 3.4 and Symfony 4.x:
And just a suggestion maybe it might be good to define the domain in Also it might be a good solution to bind a volume for root user folder in php container for persistant .bashrc, custom bin folders and global node_modules. (- ./root:/root) This is the far best container based on standard images instead of unsolicited personel images. Thanks. |
I've seen a lot of recommendations for using
|
|
I've created a Symfony 4 API Boilerplate with this repo, it still needs a little work on automation/setup but in the meantime it can be found at https://github.com/Matts/Symfony-4-Docker-API-Boilerplate |
Would you consider adding support for Symfony 3.4+ applications either on a separate branch or by way of an environment variable or some such?
The nginx configuration would need to send requests to
public/index.php
instead ofweb/app(_dev).php
, and the PHP version would need to be bumped to 7.1.3^The text was updated successfully, but these errors were encountered: