-
Notifications
You must be signed in to change notification settings - Fork 193
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 Shifter + Charliecloud profiles #826
Conversation
Codecov Report
@@ Coverage Diff @@
## dev #826 +/- ##
=======================================
Coverage 77.77% 77.78%
=======================================
Files 22 22
Lines 2529 2530 +1
=======================================
+ Hits 1967 1968 +1
Misses 562 562
Continue to review full report at Codecov.
|
indicates the necessity to add all required software to the path. This is important for pipelines with multiple environments.
as suggested by @ewels in nf-core#824 (comment)
|
||
env { | ||
PATH = "/opt/conda/bin:/opt/conda/envs/{{ cookiecutter.name_noslash }}-{{ cookiecutter.version }}/bin:$PATH" | ||
// TODO nf-core: If the pipeline uses additional environments, add them to $PATH as well |
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.
Any idea how this will work with BioContainers as used in DSL2 pipelines?
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.
Should work as the path will always be the same for Biocontainers, so the first /opt/conda/bin
should cover this nicely 👍🏻
nf_core/pipeline-template/{{cookiecutter.name_noslash}}/docs/usage.md
Outdated
Show resolved
Hide resolved
…sage.md Co-authored-by: Phil Ewels <phil.ewels@scilifelab.se>
Closes #824