Skip to content

Migrate yara/index and yara/mapper to topic channel versions#11730

Merged
SPPearce merged 1 commit into
nf-core:masterfrom
jonasscheid:yara-topic-channels
May 21, 2026
Merged

Migrate yara/index and yara/mapper to topic channel versions#11730
SPPearce merged 1 commit into
nf-core:masterfrom
jonasscheid:yara-topic-channels

Conversation

@jonasscheid
Copy link
Copy Markdown
Contributor

@jonasscheid jonasscheid commented May 21, 2026

PR checklist

  • This comment contains a description of changes (with reason).
  • If you've fixed a bug or added code that should be tested, add tests!
  • Remove all TODO statements.
  • Broadcast software version numbers to topic: versions - See version_topics
  • Follow the naming conventions.
  • Follow the input/output options guidelines.
  • Add a resource label
  • Use BioConda and BioContainers if possible to fulfil software requirements.
  • For modules:
    • nf-core modules test yara/index --profile docker
    • nf-core modules test yara/mapper --profile docker

@jonasscheid jonasscheid marked this pull request as ready for review May 21, 2026 10:08
@SPPearce SPPearce added this pull request to the merge queue May 21, 2026
Merged via the queue into nf-core:master with commit 7ad1622 May 21, 2026
36 checks passed
manascripts pushed a commit to manascripts/modules that referenced this pull request May 21, 2026
output:
tuple val(meta), path("${fasta}*") , emit: index
path "versions.yml" , emit: versions
tuple val("${task.process}"), val('yara'), eval("yara_indexer --version 2>&1 | grep 'yara_indexer version' | sed 's/^.*yara_indexer version: //; s/ .*\$//'"), topic: versions, emit: versions_yara
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('yara'), eval("yara_indexer --version 2>&1 | grep 'yara_indexer version' | sed 's/^.*yara_indexer version: //; s/ .*\$//'"), topic: versions, emit: versions_yara
tuple val("${task.process}"), val('yara'), eval("yara_indexer --version 2>&1 | sed -n 's/.*yara_indexer version: \([^ ]*\).*/\1/p"), topic: versions, emit: versions_yara

tuple val(meta), path("*.mapped.bam.bai"), emit: bai
path "versions.yml" , emit: versions
tuple val("${task.process}"), val('yara'), eval("yara_mapper --version 2>&1 | grep 'yara_mapper version' | sed 's/^.*yara_mapper version: //; s/ .*\$//'"), topic: versions, emit: versions_yara
tuple val("${task.process}"), val('samtools'), eval("samtools --version 2>&1 | head -n1 | sed 's/^.*samtools //'"), topic: versions, emit: versions_samtools
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('samtools'), eval("samtools --version 2>&1 | head -n1 | sed 's/^.*samtools //'"), topic: versions, emit: versions_samtools
tuple val("${task.process}"), val('samtools'), eval("samtools version | sed '1!d;s/.* //'"), topic: versions, emit: versions_samtools

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