Skip to content

[module knotAnnotSV] FIX: Merge mutually exclusive outputs into 1#11432

Open
tetedange13 wants to merge 2 commits intonf-core:masterfrom
tetedange13:fix-knotannotsv-out
Open

[module knotAnnotSV] FIX: Merge mutually exclusive outputs into 1#11432
tetedange13 wants to merge 2 commits intonf-core:masterfrom
tetedange13:fix-knotannotsv-out

Conversation

@tetedange13
Copy link
Copy Markdown
Contributor

@tetedange13 tetedange13 commented Apr 30, 2026

Informations

Tool knotAnnotSV can produce either HTML or Excel (.xlsm) output

But both ouput types are mutually exclusive and its nf-core module should reflect that

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.
  • Follow the naming conventions.
  • Follow the parameters requirements.
  • Follow the input/output options guidelines.
  • Add a resource label
  • Use BioConda and BioContainers if possible to fulfil software requirements.
  • Ensure that the test works with either Docker / Singularity. Conda CI tests can be quite flaky:
    • For modules:
      • nf-core modules test <MODULE> --profile docker NO DOCKER AVAILABLE
      • nf-core modules test <MODULE> --profile singularity
      • nf-core modules test <MODULE> --profile conda

@tetedange13 tetedange13 changed the title [module] FIX: Merge mutually exclusive outputs into 1 [module knotAnnotSV] FIX: Merge mutually exclusive outputs into 1 Apr 30, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant