-
Notifications
You must be signed in to change notification settings - Fork 719
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
Important! Template update for nf-core/tools v2.14.1 #1297
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,13 +1,15 @@ | ||
repository_type: pipeline | ||
nf_core_version: "2.14.1" | ||
lint: | ||
actions_ci: false | ||
files_unchanged: | ||
- assets/email_template.html | ||
- assets/email_template.txt | ||
- .gitignore | ||
- .github/PULL_REQUEST_TEMPLATE.md | ||
files_exist: | ||
- conf/modules.config | ||
actions_ci: false | ||
multiqc_config: false | ||
modules_config: false | ||
nextflow_config: | ||
- config_defaults: | ||
- params.ribo_database_manifest |
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -3,6 +3,9 @@ repos: | |
rev: "v3.1.0" | ||
hooks: | ||
- id: prettier | ||
additional_dependencies: | ||
- [email protected] | ||
|
||
- repo: https://github.com/editorconfig-checker/editorconfig-checker.python | ||
rev: "2.7.3" | ||
hooks: | ||
|
Original file line number | Diff line number | Diff line change | ||||||||
---|---|---|---|---|---|---|---|---|---|---|
|
@@ -314,6 +314,8 @@ If `-profile` is not specified, the pipeline will run locally and expect all sof | |||||||||
- A generic configuration profile to be used with [Charliecloud](https://hpc.github.io/charliecloud/) | ||||||||||
- `apptainer` | ||||||||||
- A generic configuration profile to be used with [Apptainer](https://apptainer.org/) | ||||||||||
- `wave` | ||||||||||
- A generic configuration profile to enable [Wave](https://seqera.io/wave/) containers. Use together with one of the above (requires Nextflow ` 24.03.0-edge` or later). | ||||||||||
Comment on lines
+317
to
+318
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. If this originates in the template we should update it there first. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Yep, we can update it in #1291 first so everything is updated as an example, and then push to the template. Think we have some agreement but will allow time for more discussions. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. @maxulysse when we have merged this PR can you pull in the changes from There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Done 8b0e4a5 |
||||||||||
- `conda` | ||||||||||
- A generic configuration profile to be used with [Conda](https://conda.io/docs/). Please only use Conda as a last resort i.e. when it's not possible to run the pipeline with Docker, Singularity, Podman, Shifter, Charliecloud, or Apptainer. | ||||||||||
|
||||||||||
|
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
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.
@maxulysse we need to update this in the template if we rename the profile as in #1291
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.
Indeed, I was thinking the exact same
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.
Are we 100% sure with the name?