modules icon indicating copy to clipboard operation
modules copied to clipboard

Dorothea/differentialabundance

Open Bisho2122 opened this issue 1 year ago • 2 comments

PR checklist

Closes #3085

  • [ ] This comment contains a description of changes (with reason).
  • [ ] If you've fixed a bug or added code that should be tested, add tests!
  • [X] If you've added a new tool - have you followed the module conventions in the contribution docs
  • [ ] If necessary, include test data in your PR.
  • [X] Remove all TODO statements.
  • [X] Emit the versions.yml file.
  • [X] Follow the naming conventions.
  • [] Follow the parameters requirements.
  • [X] Follow the input/output options guidelines.
  • [ ] Add a resource label
  • [X] 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
      • [ ] nf-core modules test <MODULE> --profile singularity
      • [ ] nf-core modules test <MODULE> --profile conda
    • For subworkflows:
      • [ ] nf-core subworkflows test <SUBWORKFLOW> --profile docker
      • [ ] nf-core subworkflows test <SUBWORKFLOW> --profile singularity
      • [ ] nf-core subworkflows test <SUBWORKFLOW> --profile conda

Bisho2122 avatar Mar 19 '24 14:03 Bisho2122

@pinin4fjords, I'll add the nf-tests and rest of other tests tomorrow. Let me know if you need me to add a few lines in the differentialabundance workflow. Till then, if you have any comments on the files, let me know.

Bisho2122 avatar Mar 19 '24 15:03 Bisho2122

@Bisho2122 , do you need any help with this?

SPPearce avatar Sep 10 '24 15:09 SPPearce

@pinin4fjords @Bisho2122 Are you still on this or can we close the PR for now? :)

famosab avatar Mar 11 '25 09:03 famosab

Let's close for now, and reopen if work resumes

pinin4fjords avatar Mar 12 '25 15:03 pinin4fjords