Skip to content

Improve docs to include simple 2-liner to run Tox using Docker. - #15

Open
wjlow wants to merge 1 commit into
themattrix:masterfrom
wjlow:patch-1
Open

wjlow wants to merge 1 commit into
themattrix:masterfrom
wjlow:patch-1

Conversation

@wjlow

@wjlow wjlow commented Apr 4, 2018

Copy link
Copy Markdown

No description provided.

@MarSoft

MarSoft commented Jun 15, 2019

Copy link
Copy Markdown

I think we should not mount directory to /app but rather mount it to /src. The /docker-entrypoint.sh script copies everything except for dot directories from /src to /app, then it chowns /app and runs tox over it. We don't want to chown in our real filesytstem (and sometimes we won't be able to do it anyway), so we want to mount our source to /src instead.

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