Minor updates to biscuit modules

[njspix]

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!
• 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:
  • PROFILE=docker pytest --tag <MODULE> --symlink --keep-workflow-wd --git-aware
  • PROFILE=singularity pytest --tag <MODULE> --symlink --keep-workflow-wd --git-aware
  • PROFILE=conda pytest --tag <MODULE> --symlink --keep-workflow-wd --git-aware

Minor updates to these modules:

• version bumps for biscuit and samtools
• multi-package container update (corresponding)
• add set -o pipefail to all commands involving pipes
• add --write-index option to align
• add an output channel to both align and biscuitblaster (indexed_bam) containing both .bam and .bai files

[njspix]

Note that biscuitblaster tests are expected to fail until nf-core/test-datasets#715 is merged. Incorrect read naming makes samblaster puke, but I wasn't catching this before because set -o pipefail wasn't set...

[muffato]

add an output channel to both align and biscuitblaster (indexed_bam) containing both .bam and .bai files

Having the bam files in two output channels would be the first time ever in modules ! Let me ask on Slack what the other maintainers think about this

[njspix]

Ok, fixed those issues. Thanks for your input and please let me know if there is anything else I can do!

[muffato]

Looks good to me, thank you !