Skip to content

Swap verifybamid to topic#12262

Merged
SPPearce merged 7 commits into
masterfrom
verifybamid_topic
Jul 14, 2026
Merged

Swap verifybamid to topic#12262
SPPearce merged 7 commits into
masterfrom
verifybamid_topic

Conversation

@SPPearce

@SPPearce SPPearce commented Jul 7, 2026

Copy link
Copy Markdown
Contributor

Swap verifybamid to topic

@github-actions github-actions Bot added the size/m label Jul 7, 2026
@SPPearce
SPPearce marked this pull request as ready for review July 7, 2026 12:57
"id": "test",
"single_end": false
},
"test.depthRG:md5,2374504c4138c04f3b29210dcb3da96f"

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I am not sure why this one is removed?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

This wasn't being generated because ext.args wasn't defined in the main.nf, meaning the nextflow config which was telling it to not be generated was ignored.
I've just removed the config entirely, the file isn't empty so everything works smoothly.

tuple val(meta), path("*.depthRG"), emit: depthrg, optional:true
tuple val(meta), path("*.bestSM") , emit: bestsm , optional:true
tuple val(meta), path("*.bestRG") , emit: bestrg , optional:true
tuple val("${task.process}"), val('verifybamid'), eval("echo '1.1.3'"), emit: versions_verifybamid, topic: versions

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
tuple val("${task.process}"), val('verifybamid'), eval("echo '1.1.3'"), emit: versions_verifybamid, topic: versions
tuple val("${task.process}"), val('verifybamid'), val('1.1.3'), emit: versions_verifybamid, topic: versions

@SPPearce
SPPearce added this pull request to the merge queue Jul 14, 2026
Merged via the queue into master with commit 2e03e33 Jul 14, 2026
129 checks passed
@SPPearce
SPPearce deleted the verifybamid_topic branch July 14, 2026 05:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants