Skip to content

Patch release v3.0.1#192

Merged
Aratz merged 3 commits into
nf-core:masterfrom
Aratz:patch
Mar 9, 2026
Merged

Patch release v3.0.1#192
Aratz merged 3 commits into
nf-core:masterfrom
Aratz:patch

Conversation

@Aratz

@Aratz Aratz commented Mar 9, 2026

Copy link
Copy Markdown
Contributor

If-statements in withName blocks cause errors when their condition doesn't write anything to the config.

Besides, they are deprecated in Nextflow's upcoming strict syntax.

Fixes #191

PR checklist

  • This comment contains a description of changes (with reason).
  • Make sure your code lints (nf-core pipelines lint).
  • Ensure the test suite passes (nextflow run . -profile test,docker --outdir <OUTDIR>).
  • CHANGELOG.md is updated.

@Aratz Aratz self-assigned this Mar 9, 2026
@Aratz Aratz changed the title Remove if-statements from modules.config Patch release v3.0.1 Mar 9, 2026
@Aratz Aratz marked this pull request as ready for review March 9, 2026 09:51
@Aratz Aratz requested a review from a team as a code owner March 9, 2026 09:51

@johandahlberg johandahlberg left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@Aratz Aratz requested a review from fbdtemme March 9, 2026 10:27
@Aratz Aratz merged commit 9c79407 into nf-core:master Mar 9, 2026
37 of 41 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Pipeline fails when --experiment_summary_container is not set

3 participants