-
-
Notifications
You must be signed in to change notification settings - Fork 18
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
Add some information on the docker --workdir option #25
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The codeblocks are using more traditional markdown syntax instead of the lesson codeblock syntax. <- this needs to be fixed before merging
Also broke some lines. <- this is mostly style
Co-authored-by: Sarah Stevens <[email protected]>
Many thanks for the review and suggestions, @sstevens2, and pointing out the use of the wrong syntax for the codeblocks, I didn't notice that when writing the updated content. Suggestions commited. |
This PR aims to address issue #24 by adding a callout box to the end of episode 6 (06-singularity-images-prep.md) describing how the
--workdir
or-w
Docker command line option can help to simplifysingularity
command lines when using the Docker Singularity container (and especially when using an alias to run the container).I'd appreciate some feedback from the reviewers on whether the new text is sufficiently clear - this concept was a little challenging to explain clearly so any suggestions on how the text might be improved are welcome.