bumped MultiQC version for dumpsoftwareversions

[harper357]

PR checklist

This just updates dumpsoftwareversions to the latest version of theMultiQC container.

• [x] This comment contains a description of changes (with reason).
• [ ] Emit the versions.yml file.
• [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

Running the test fails because the versions are updated in the container, I am not sure how you usually go about updating the test.

[jfy133]

I wonder if this could also be replaced with the ubuntu container... I wonder if that has a basic python installation?

[harper357]

I tried to run the ubuntu container but python is not installed in it

[jfy133]

@maxulysse pointed out that we don't use this anymore, but instead with a function in the template. He wondered if we should deprecate it.

[SPPearce]

So, do we want to deprecate this?

[jfy133]

I don't know if we have a deprecation procedure do we?

[SPPearce]

I don't know if we have a deprecation procedure do we?

Oh James, I thought you knew the guidelines 😉.

[SPPearce]

Looks like we have 3 modules and a subworkflow deprecated so far.

[jfy133]

I don't know if we have a deprecation procedure do we?

Oh James, I thought you knew the guidelines 😉.

OH nice! Then yes, lets deprecate :)

[SPPearce]

Added a message. I don't know what happens to the tests at that point, but they seem to be fine for the other modules so maybe they just ignore the assert?

[famosab]

@jfy133 @SPPearce @harper357 Can we finish this PR somehow?