Fix: arriba: rename argument and reorder alphabetically

[rannick]

PR checklist

Rename argument to fix language server issues, use wave containers

• 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
  • For subworkflows:
    • nf-core subworkflows test <SUBWORKFLOW> --profile docker
    • nf-core subworkflows test <SUBWORKFLOW> --profile singularity
    • nf-core subworkflows test <SUBWORKFLOW> --profile conda

[GallVp]

This should not be needed now. The issues with the CI runners have been resolved. Can you kindly remove this and retry the CI tests.

[GallVp]

Same as above! Thank you!

[GallVp]

Looks good. Thank you for the contribution.

[atrigila]

How is genome used in the wget command? Perhaps we could pass the path to the arriba database. Therefore, if the datbaase path changes or is updated (e.g. arriba_v2.5.0), we don't have to update the hardcoded https://github.com/suhrig/arriba/releases/download/v2.4.0/arriba_v2.4.0.tar.gz -O arriba_v2.4.0.tar.gz in the module, we just have to pass a new val(arriba_database).

So then, it would be: wget $arriba_database