Skip to content

Add manifest.diagram to nextflow.config - #313

Merged
LouisLeNezet merged 2 commits into
nf-core:devfrom
LouisLeNezet:metro_maps
Sep 22, 2026
Merged

LouisLeNezet merged 2 commits into
nf-core:devfrom
LouisLeNezet:metro_maps

Conversation

@LouisLeNezet

@LouisLeNezet LouisLeNezet commented Sep 22, 2026

Copy link
Copy Markdown
Collaborator

Adds the new manifest.diagram config attribute, pointing at the pipeline overview diagram.

See nf-core/tools#4460 for background: manifest.diagram gives a canonical location for the metro map, so that downstream consumers can find it. We'll use it for the nf-core website soon, for example. The new config option won't be released until Nextflow 26.10, but older versions ignore unknown manifest fields, so it's safe to set in any pipeline today.

Duplicate of #309

This PR also:

  • Update author and contributors manifest fields
  • Remove compute_freq unused parameter from tool_bibliography (as vcflib fix has been removed and replaced by bcftools)

@github-actions

github-actions Bot commented Sep 22, 2026

Copy link
Copy Markdown

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

Posted for pipeline commit f29edc2

+| ✅ 215 tests passed       |+
#| ❔   2 tests were ignored |#
!| ❗   1 tests had warnings |!
Details

❗ Test warnings:

  • pipeline_todos - TODO string in CHANGELOG.md: ## v1.2.0dev - [unreleased replace with date on release ]

❔ Tests ignored:

✅ Tests passed:

Run details

  • nf-core/tools version 4.1.0
  • Run at 2026-09-22 10:16:39

@LouisLeNezet LouisLeNezet self-assigned this Sep 22, 2026

@piplus2 piplus2 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM

@LouisLeNezet
LouisLeNezet merged commit e7252e1 into nf-core:dev Sep 22, 2026
21 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