-
Notifications
You must be signed in to change notification settings - Fork 32
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
Changes to Docker material #270
Merged
Merged
Changes from 1 commit
Commits
Show all changes
23 commits
Select commit
Hold shift + click to select a range
f526e98
Remove MAINTAINER label from Dockerfiles
fasterius 99795d2
Add note on OCI Docker label best practices
fasterius 31bff6c
Fix naming of Dockerfiles according to conventions
fasterius c215721
Harmonise Dockerfile formatting
fasterius 19e9a0f
Fix naming of `linux/amd64` platform specification
fasterius c32405b
Add missing note title word
fasterius a58e880
Streamline `[slim/conda].Dockerfile` files
fasterius a5b23f3
Streamline final tutorial Dockerfile
fasterius d17a01a
Add extra material for multi-stage Docker builds
fasterius 1682dd0
Fix broken Docker image
fasterius bcd97a5
Minor updates to conform to build checks
johnne 461337b
Fix CMD line of Dockerfile
johnne c6cca74
Add SHELL instruction
johnne 0994714
Fix embedding of resources in Quarto document
fasterius 9324734
Set SHELL to not require JSON format for CMD
johnne e3e8b14
Fix embed-resources
johnne a1668fe
Move callout for ARM users
johnne d74046a
Merge branch 'docker-changes' of github.com:NBISweden/workshop-reprod…
johnne 837c4c3
Remove section on jupyter notebook with Docker
johnne 1fc4dad
Remove expose part from Dockerfile
johnne 11805cf
Make supplementary table less redundant
johnne 9365c3b
Fix minor formatting and spelling errors
johnne 61d1b4a
Give explicit name for multi-stage Dockerfile
johnne File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
I modified the code here to make the sample table less cluttered. There used to be 2 lines for each sample due to the info in
gsm$characteristics_ch1
. Now we extract thetreatment:
andgrowth phase
info and put in separate columns, resulting in a condensed table.