Skip to content

Strict-syntax compliance: Nextflow 25.10.4, nf-schema 2.6.1#1680

Merged
pinin4fjords merged 9 commits into
devfrom
update-nf-schema-strict-syntax
May 11, 2026
Merged

Strict-syntax compliance: Nextflow 25.10.4, nf-schema 2.6.1#1680
pinin4fjords merged 9 commits into
devfrom
update-nf-schema-strict-syntax

Conversation

@pinin4fjords
Copy link
Copy Markdown
Member

@pinin4fjords pinin4fjords commented Jan 23, 2026

Summary

Brings nf-core/rnaseq to a 0-warning state under the Nextflow v2 strict-syntax parser, and closes the local-module documentation gap that was causing repeated meta.yml warnings from nf-core tooling.

  • Bump nextflowVersion floor to !>=25.10.4 (matches the current nf-core template) and align the CI matrix (nf-test.yml, nf-test-arm.yml, nf-test-gpu.yml), README badge, and ro-crate metadata.
  • Bump nf-schema from 2.5.1 to 2.6.1. The 2.6.x line is only published to the new Seqera Plugin Registry, which Nextflow 25.10+ consults; the Nextflow bump is the prerequisite for resolving it. Adds support for validation.defaultIgnoreParams and validation.monochromeLogs under the v2 parser.
  • Resolve the strict-syntax lint warnings flagged in the strict-syntax-health report:
    • Local subworkflows (multiqc_rnaseq, prepare_genome_indices, prepare_genome_references): replace implicit it closure params with named ones; prefix unused meta parameters with _.
    • Resync 6 nf-core components to upstream master so the previously-merged strict-syntax fixes land via proper SHA bumps in modules.json (no transitive cascade):
      • modules/nf-core/custom/gtffilter (also picks up apptainer support and the fully-qualified quay.io/biocontainers/... container URI)
      • modules/nf-core/ea-utils/gtf2bed (same)
      • subworkflows/nf-core/bam_dedup_umi
      • subworkflows/nf-core/bam_markduplicates_picard
      • subworkflows/nf-core/bam_qc_rnaseq
      • subworkflows/nf-core/fastq_qc_trim_filter_setstrandedness
  • Add missing meta.yml for the local DESEQ2_QC and PREPROCESS_TRANSCRIPTS_FASTA_GENCODE modules so the repeated WARNING Could not find meta.yml for ... lines no longer appear during nf-core modules/subworkflows invocations.

Test plan

  • CI matrix green on 25.10.4 and latest-everything.
  • NXF_SYNTAX_PARSER=v2 nextflow run nf-core/rnaseq --help passes (already shown green on the strict-syntax-health report against Nextflow 26.04.0).
  • nf-core modules lint no longer emits the Could not find meta.yml warnings for DESEQ2_QC / PREPROCESS_TRANSCRIPTS_FASTA_GENCODE.

Update nf-schema plugin from 2.5.1 to 2.6.1 to fix unrecognized config
warnings (`validation.defaultIgnoreParams`, `validation.monochromeLogs`)
when running with the Nextflow v2 strict syntax parser.

This is part of the effort to achieve strict syntax compliance for the
`--help` check in nf-core/strict-syntax-health.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@github-actions
Copy link
Copy Markdown

This PR is against the master branch ❌

  • Do not close this PR
  • Click Edit and change the base to dev
  • This CI test will remain failed until you push a new commit

Hi @pinin4fjords,

It looks like this pull-request is has been made against the nf-core/rnaseq master branch.
The master branch on nf-core repositories should always contain code from the latest release.
Because of this, PRs to master are only allowed if they come from the nf-core/rnaseq dev branch.

You do not need to close this PR, you can change the target branch to dev by clicking the "Edit" button at the top of this page.
Note that even after this, the test will continue to show as failing until you push a new commit.

Thanks again for your contribution!

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jan 23, 2026

nf-core pipelines lint overall result: Passed ✅ ⚠️

Posted for pipeline commit 802f166

+| ✅ 215 tests passed       |+
#| ❔  19 tests were ignored |#
!| ❗   7 tests had warnings |!
Details

❗ Test warnings:

  • pipeline_todos - TODO string in awsfulltest.yml: You can customise AWS full pipeline tests as required
  • pipeline_todos - TODO string in nextflow.config: Specify any additional parameters here
  • pipeline_todos - TODO string in methods_description_template.yml: #Update the HTML below to your preferred methods description, e.g. add publication citation for this pipeline
  • pipeline_todos - TODO string in main.nf: Optionally add in-text citation tools to this list.
  • pipeline_todos - TODO string in main.nf: Optionally add bibliographic entries to this list.
  • pipeline_todos - TODO string in main.nf: Only uncomment below if logic in toolCitationText/toolBibliographyText has been filled!
  • pipeline_todos - TODO string in base.config: Check the defaults for all processes

❔ Tests ignored:

  • files_exist - File is ignored: conf/modules.config
  • files_exist - File is ignored: conf/containers_conda_lock_files_amd64.config
  • files_exist - File is ignored: conf/containers_conda_lock_files_arm64.config
  • files_exist - File is ignored: conf/containers_docker_amd64.config
  • files_exist - File is ignored: conf/containers_docker_arm64.config
  • files_exist - File is ignored: conf/containers_singularity_https_amd64.config
  • files_exist - File is ignored: conf/containers_singularity_https_arm64.config
  • files_exist - File is ignored: conf/containers_singularity_oras_amd64.config
  • files_exist - File is ignored: conf/containers_singularity_oras_arm64.config
  • nextflow_config - Config default ignored: params.ribo_database_manifest
  • nf_test_content - nf_test_content
  • files_unchanged - File ignored due to lint config: assets/email_template.html
  • files_unchanged - File ignored due to lint config: assets/email_template.txt
  • files_unchanged - File ignored due to lint config: assets/nf-core-rnaseq_logo_light.png
  • files_unchanged - File ignored due to lint config: docs/images/nf-core-rnaseq_logo_dark.png
  • files_unchanged - File ignored due to lint config: .gitignore or .prettierignore
  • actions_nf_test - actions_nf_test
  • modules_config - modules_config
  • container_configs - container_configs

✅ Tests passed:

Run details

  • nf-core/tools version 4.0.2
  • Run at 2026-05-11 11:05:20

@pinin4fjords pinin4fjords changed the base branch from master to dev January 23, 2026 19:21
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
pinin4fjords and others added 3 commits May 11, 2026 09:36
Bumps `nextflowVersion` from `!>=25.04.3` to `!>=25.10.4` so the
Seqera Plugin Registry resolves `nf-schema` 2.6.1 (deliberately not
published to the legacy nextflow-io/plugins index, per
nextflow-io/plugins#197). CI matrix, README
badge, and ro-crate metadata follow.

Silences the strict-syntax v2-parser warnings flagged in the
nf-core/strict-syntax-health report:
- local subworkflows: replace implicit `it` closure params with named
  ones; prefix unused `meta` parameters with `_`
- nf-core subworkflows: carry the upstream-merged fixes for
  bam_dedup_umi, bam_markduplicates_picard, bam_qc_rnaseq, and
  fastq_qc_trim_filter_setstrandedness so CI exercises them now;
  proper modules.json resync to follow once the nf-core/modules
  changes for gtffilter and gtf2bed land.

Two `args` warnings on the gtffilter/gtf2bed templated modules remain
deferred to the upstream nf-core/modules PR.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@pinin4fjords pinin4fjords changed the title chore: Update nf-schema to 2.6.1 for strict syntax compatibility Strict-syntax compliance: Nextflow 25.10.4, nf-schema 2.6.1 May 11, 2026
pinin4fjords and others added 3 commits May 11, 2026 10:54
Bump modules.json SHAs and refresh on-disk files for the components
whose strict-syntax fixes were previously carried inline:

- modules:     custom/gtffilter, ea-utils/gtf2bed
- subworkflows: bam_dedup_umi, bam_markduplicates_picard,
                bam_qc_rnaseq, fastq_qc_trim_filter_setstrandedness

Subworkflow main.nf files were already byte-equivalent with master
after the prior inline edits, so only the SHA pins change for those.
The two module main.nf files also pick up apptainer container-engine
support and fully-qualified quay.io biocontainer URIs from upstream
along with the strict-syntax `args` workaround.

modules.json is touched as six single-line SHA updates rather than the
full reformat nf-core/tools 4.0.2's `install --force` would produce.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
…s_transcripts_fasta_gencode

Silences the repeated `WARNING Could not find meta.yml for ...`
emissions seen during every `nf-core modules`/`subworkflows`
invocation, and documents inputs/outputs/tool versions in the
schema-validated nf-core meta-schema format.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Comment thread CHANGELOG.md Outdated
Comment thread modules/local/deseq2_qc/meta.yml
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Comment thread CHANGELOG.md Outdated
@pinin4fjords pinin4fjords enabled auto-merge May 11, 2026 11:04
@pinin4fjords pinin4fjords merged commit 8489f01 into dev May 11, 2026
83 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.

2 participants