Skip to content

Conversation

@yurikoster1
Copy link

…-compose; use gitignore.io; Updated app Dockerfile

Moved laravel app to www folder;
Updated docker-compose version to 3.7;
Add vscode and phpstorm+all from gitignore.io to .gitignore;
Separated app's Dockerfile into 2 different stages to allow future use of a single Dockerfile in both development and production;
In app's Dockerfile dev stage added the entrypont script and the .env file to be used as a template with dockerize;

…o; Updated app Dockerfile

Moved laravel app to www folder;
Updated docker-compose version to 3.7;
Add vscode and phpstorm+all from gitignore.io to .gitignore; 
Separated app's Dockerfile into 2 different stages to allow future use of a single Dockerfile in both development and production;
In app's Dockerfile dev stage added the entrypont script and the .env file to be used as a template with dockerize;
@argentinaluiz argentinaluiz changed the base branch from master to more_organized August 31, 2020 10:10
@argentinaluiz argentinaluiz merged commit 0b958eb into codeedu:more_organized Aug 31, 2020
@argentinaluiz
Copy link
Member

Thanks @yurikoster1 . Excellent job!

Your pull request was moved to new branch, because the course used the old structure, but I will recommend it to our students.

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.

2 participants