Skip to content

Conversation

@pinin4fjords
Copy link
Member

Summary

Converts 5 modules from legacy versions.yml file emission to topic: versions channel emission (batch 2, following #9796).

Modules converted:

  • preseq/lcextrap
  • qualimap/rnaseq
  • star/genomegenerate (3 tools: star, samtools, gawk)
  • subread/featurecounts
  • sylphtax/taxprof

Changes

  • Replace path "versions.yml", emit: versions with tuple val("${task.process}"), val('toolname'), eval('...'), emit: versions_toolname, topic: versions
  • Remove cat <<-END_VERSIONS heredoc blocks from script and stub sections
  • Update test assertions to use process.out.findAll { key, val -> key.startsWith('versions') }
  • Update meta.yml with new output structure

Test plan

  • All module tests pass with nf-test test --profile docker
  • Snapshots updated with structured version data

🤖 Generated with Claude Code

pinin4fjords and others added 5 commits February 2, 2026 14:33
Convert the following modules from legacy versions.yml file emission
to modern topic: versions channel emission:

- preseq/lcextrap
- qualimap/rnaseq
- star/genomegenerate
- subread/featurecounts
- sylphtax/taxprof

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Use head -1 to get first line only, avoiding multi-line parsing issues.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Remove echo $(...) pattern which can cause issues.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Use tail -1 to get only the version, not the config warning.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Commands containing | and : need to be quoted in YAML.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@mashehu
Copy link
Contributor

mashehu commented Feb 2, 2026

please split them up. it makes it easier to review if it is always the same version and version command

@pinin4fjords
Copy link
Member Author

please split them up. it makes it easier to review if it is always the same version and version command

OK, fine, just let me get them passing here and I'll do it

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@pinin4fjords
Copy link
Member Author

Splitting into separate PRs per reviewer request: #9827 #9828 #9829 #9830 #9831

@pinin4fjords pinin4fjords deleted the feat/topics-version-batch2 branch February 2, 2026 15:10
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.

2 participants