-
Notifications
You must be signed in to change notification settings - Fork 2
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
dropped quay.io from docker paths #94
Conversation
This PR is against the
|
|
Included the override container registry process (for more information see phac-nml pipeline standards) to allow the container directives to have a conditional check in the nested ternary operator. Added this to the |
Added the default container registries to |
Resolving the branch conflict included updating the |
…ault container option
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.
This looks great. Thanks so much for your work on this Steven and making all those changes/testing things out 😄
I just have one comment.
Just looked everything over! It looks great and thank you so much for your help! |
In addition to updating containers to use a default registry python3 {
singularity = "biocontainers/staramr:0.10.0--pyhdfd78af_0"
docker = "biocontainers/staramr:0.10.0--pyhdfd78af_0"
} Which has Python 3.11.4 running under the hood. So will replace with Python 3.12 (the closest quay.io container) |
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.
Looks good. Thanks so much Steven 😄
Removed docker
quay.io
tags.