Skip to content

This should allow the dockerfile to run properly in linux#122

Open
john-potts wants to merge 6 commits intomainfrom
docker_compose_linux_fix
Open

This should allow the dockerfile to run properly in linux#122
john-potts wants to merge 6 commits intomainfrom
docker_compose_linux_fix

Conversation

@john-potts
Copy link
Copy Markdown

user and volume additions address permissions, depends_on should address a race condition

Comment thread docker-compose.yaml Outdated
- "./:/app/"
- /app/vendor # Use dependencies installed by composer during image building.
# Remove the above line to use the local `vendor` directory.
- /app
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why map /app twice?

@john-potts john-potts force-pushed the docker_compose_linux_fix branch from 44bc83a to cfe69f9 Compare March 26, 2026 20:17
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