Skip to content

chore: template update to nf-core/tools 4.0.2#150

Open
znorgaard wants to merge 1 commit into
devfrom
zn_template_4.0.2
Open

chore: template update to nf-core/tools 4.0.2#150
znorgaard wants to merge 1 commit into
devfrom
zn_template_4.0.2

Conversation

@znorgaard
Copy link
Copy Markdown
Collaborator

Updates the nf-core template from 3.5.2 → 4.0.2 via a fresh nf-core pipelines sync. Supersedes the stale automated PR #149 (its merge branch predated recent dev work).

Notable 4.0 changes applied

  • nextflowVersion!>=25.10.4; added the wave profile
  • Webhook notifications removed: hook_url, assets/slackreport.json, assets/adaptivecard.json, and the imNotification call (the dormant helper remains in the kept nf-core utils_nfcore_pipeline and is no longer used)
  • Added a workflow.onError handler
  • CONTRIBUTING.md moved .github/docs/
  • git hooks switched to prek; .lineage/ added to .gitignore
  • nextflow_schema.json regenerated (all custom params + descriptions preserved)
  • RO-Crate metadata description synced to the real README

Preserved fastquorum customizations

  • Tuned process resources, igenomes CHM13 entry, nf-fgbio plugin
  • All samplesheet validation (library_id/lane/flowcell, fgbio read-structure checks), non-random UMI support, fgbio modules
  • Real Zenodo DOIs, contributors, and pipeline docs

Scope notes

  • nf-core modules kept at current versions (fastqc/multiqc not bumped) to avoid the 4.0 MultiQC input-signature change; a dedicated nf-core modules update is a sensible follow-up.
  • conf/containers_*.config removed: the 4.0 auto-generated container configs can't be regenerated for this pipeline's modules (local fgbio modules have no meta.yml; kept nf-core modules predate the meta.yml containers: block). The pipeline uses per-module container directives as before; configs can be added when modules are updated. No lint rule was disabled.

Verification

  • nf-core pipelines lint — 258 passed, 0 failed (14 warnings: newer-version-available + local-subworkflow meta_yml, all pre-existing)
  • prek run --all-files — prettier, whitespace, Nextflow Lint all pass
  • nf-test test --profile debug,test,docker26/26 passed, no snapshot changes

Note: the automated PR #149 (nf-core-template-merge-4.0.2) is superseded by this PR and can be closed, along with its branch.

🤖 Generated with Claude Code

@znorgaard znorgaard force-pushed the zn_template_4.0.2 branch from d8d8447 to e365576 Compare June 5, 2026 18:28
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jun 5, 2026

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

Posted for pipeline commit 20f1d80

+| ✅ 209 tests passed       |+
#| ❔  10 tests were ignored |#
!| ❗   3 tests had warnings |!
Details

❗ Test warnings:

  • files_unchanged - LICENSE does not match the template
  • pipeline_todos - TODO string in awsfulltest.yml: You can customise AWS full pipeline tests as required
  • pipeline_todos - TODO string in CONTRIBUTING.md: Add any pipeline specific contribution guidelines here, such as coding styles, procedures, checklists etc.

❔ Tests ignored:

  • files_exist - File is ignored: .github/workflows/awsfulltest.yml
  • files_exist - File is ignored: .github/workflows/awstest.yml
  • files_exist - File is ignored: conf/modules/modules.config
  • files_unchanged - File ignored due to lint config: .github/PULL_REQUEST_TEMPLATE.md
  • files_unchanged - File ignored due to lint config: .github/workflows/branch.yml
  • files_unchanged - File ignored due to lint config: .github/workflows/linting.yml
  • files_unchanged - File ignored due to lint config: assets/nf-core-fastquorum_logo_light.png
  • files_unchanged - File ignored due to lint config: docs/images/nf-core-fastquorum_logo_dark.png
  • files_unchanged - File ignored due to lint config: .gitignore or .prettierignore
  • template_strings - template_strings

✅ Tests passed:

Run details

  • nf-core/tools version 4.0.2
  • Run at 2026-06-05 19:24:17

@znorgaard znorgaard marked this pull request as ready for review June 5, 2026 18:47
@znorgaard znorgaard force-pushed the zn_template_4.0.2 branch from e365576 to 09f4a12 Compare June 5, 2026 18:49
Comment thread .github/CONTRIBUTING.md Outdated
Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Moved to docs/CONTRIBUTING.md

Update the nf-core template from 3.5.2 to 4.0.2 via a fresh
`nf-core pipelines sync`.

4.0 changes applied:
- nextflowVersion -> !>=25.10.4; add the wave profile
- remove webhook notifications (hook_url, slackreport.json,
  adaptivecard.json, imNotification call); add workflow.onError handler
- move CONTRIBUTING.md from .github/ to docs/
- switch git hooks to prek; ignore .lineage/
- regenerate nextflow_schema.json (custom params + descriptions preserved)
- sync ro-crate-metadata.json description with the README

Preserved fastquorum customizations: tuned process resources, igenomes
CHM13, nf-fgbio plugin, samplesheet/UMI validation, fgbio modules,
DOIs/contributors/docs.

nf-core modules kept at current versions (fastqc/multiqc not bumped) to
avoid the 4.0 MultiQC input-signature change; module updates are a
separate follow-up. conf/containers_*.config removed: the 4.0
auto-generated container configs cannot be regenerated for this
pipeline's modules (local fgbio modules lack meta.yml; kept nf-core
modules predate the meta.yml containers block).

Verified: nf-core pipelines lint (258 passed, 0 failed), prek hooks,
nf-test --profile debug,test,docker (26/26 passed).

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@znorgaard znorgaard force-pushed the zn_template_4.0.2 branch from 09f4a12 to 20f1d80 Compare June 5, 2026 19:22
@znorgaard znorgaard requested review from SPPearce and nh13 June 5, 2026 19:23
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.

3 participants