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

Mount the workspace and set as default workdir #24

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

ldaneliukas
Copy link

@ldaneliukas ldaneliukas commented Jul 9, 2021

Similarly to #23 , when running containers directly, they will automatically mount the Github workspace and set it as the workdir for the container, so that all of the repository contents could be accessed (assuming a checkout was done).

This seems like a safe default and it'd be intuitive to keep this flow when running containers via the docker-run-action.

Changes:

  • Automatically mount the $GITHUB_WORKSPACE
  • Ability to specify the workdir for the container. Defaults to $GITHUB_WORKSPACE

Note, this change depends on #23 , since the container we are running needs to know about the Github environment variables in order to mount the correct volume for the workspace. As the action is now, we are ignoring all Github and user-defined env vars (set via env: on the step).

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.

1 participant