-
Notifications
You must be signed in to change notification settings - Fork 135
Update to nextflow strict syntax #872
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
Conversation
@nf-core-bot fix linting |
Unfortunately all |
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 to me
The fix is to downgreat [email protected] to [email protected] in the |
Thanks! |
Nextflow is changing eventually to stricter syntax, described in https://www.nextflow.io/docs/latest/strict-syntax.html. This PR solves all errors (but not warnings) regarding this strict syntax for Nextflow extension for Visual Studio Code, version 1.2.0.
Following this stricter syntax allows using the full potential of the Nextflow language support plugin and it should make the pipeline code (more) future proof.
Specific changes:
when
statement removed (this causes most of the file changed here)for
loops were replaced with.each
.anyMatch
was replaced with.any
def
addedswitch
was replaced byif/else
(only in nf-core module vsearch/usearchglobal ) via Update vsearch/usearchglobal to strict syntax modules#8333workflows/ampliseq.nf
had the params & channel creation moved intoworkflow
it
was added asit ->
in all local workflows and subworkflows [this is not causing an error, butit
alone is deprecated]There are no functional changes whatsoever in this PR, only syntax improvements.
PR checklist
nf-core pipelines lint
).nextflow run . -profile test,docker --outdir <OUTDIR>
).nextflow run . -profile debug,test,docker --outdir <OUTDIR>
).docs/usage.md
is updated.docs/output.md
is updated.CHANGELOG.md
is updated.README.md
is updated (including new tool citations and authors/contributors).