Skip to content

Add stub block and migrate pairix to topic channels#11437

Open
HReed1 wants to merge 2 commits intonf-core:masterfrom
HReed1:stub-topics-pairix
Open

Add stub block and migrate pairix to topic channels#11437
HReed1 wants to merge 2 commits intonf-core:masterfrom
HReed1:stub-topics-pairix

Conversation

@HReed1
Copy link
Copy Markdown
Contributor

@HReed1 HReed1 commented Apr 30, 2026

Changes

  • Migrated version emission from versions.yml heredoc to topic: versions tuple pattern
  • Added stub: block with appropriate touch outputs
  • Added stub test case
  • Simplified snapshot assertions to snapshot(process.out).match()
  • Auto-fixed meta.yml to align eval expressions with main.nf

Verification

  • nf-core modules lint pairix: 52/52 passed, 0 failures
  • nf-test (Docker): 2/2 passed, snapshots stable across consecutive runs

Part of #4570

Copy link
Copy Markdown
Contributor

@SPPearce SPPearce left a comment

Choose a reason for hiding this comment

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

Your version extraction is not working.
It does not need echo.
Please add a check step in your submission instructions that verifies the version is being extracted correctly

[
"PAIRIX",
"pairix",
"Program: pairix (PAIRs file InderXer)\n0.3.7\n\n\n\nOptions: -p STR preset: pairs, merged_nodups, old_merged_nodups, gff, bed, sam, vcf, psltbl [gff]\n -s INT sequence name column [1]\n -d INT second sequence name column [null]\n -b INT start1 column [4]\n -e INT end1 column; can be identical to '-b' [5]\n -u INT start2 column [null]\n -v INT end2 column; can be identical to '-u' [null or identical to the start2 specified by -u]\n -T delimiter is space instead of tab.\n -L query region is not a string but a file listing query regions\n -S INT skip first INT lines [0]\n -c CHAR symbol for comment/meta lines [#]\n -w CHAR symbol for query region separator [|]\n -r FILE replace the header with the content of FILE [null]\n -0 zero-based coordinate\n -h print also the header lines\n -H print only the header lines\n -B print only the number of bgzf blocks for existing chromosome (pairs)\n -W print only the region split character\n -Y Only check if the file is a triangle (i.e. a chromosome pair occurs only in one direction (e.g. if chr1|chr2 exists, chr2|chr1 doesn't))\n -l list chromosome names\n -n print only the total line count (same as gunzip -c | wc -l but much faster)\n -f force to overwrite the index\n -a autoflip query when the matching chromosome pair doesn't exist\n --help print usage with exit 0"
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

The version should be here instead.

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