New Module: NACHO_NORMALISE

[alanmmobbs93]

PR checklist

Closes Nanostring #111

• This comment contains a description of changes (with reason).
• If you've fixed a bug or added code that should be tested, add tests!
• 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.
• Remove all TODO statements.
• Emit the versions.yml file.
• 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
    • nf-core modules test <MODULE> --profile singularity
    • nf-core modules test <MODULE> --profile conda

Do you know who wrote the original local module for the Nanostring? So we can give propper credit

[nschcolnicov]

Very nice!

[atrigila]

Good job :) made a few suggestions according to guidelines

[atrigila]

You have a few linting issues that you can solve checking: nf-core modules lint nacho/normalise

[nschcolnicov]

Better to have this added into the tests/nextflow.config file and then adding the line "config "./nextflow.config" to the main.nf.test file. Just because the docs https://www.nextflow.io/docs/latest/module.html#module-binaries say to add them in the config or pipeline file, and also because no other module seems to set this type of parameters directly in the main.nf file

[nschcolnicov]

Thank you, looks good to me!