Update the untar module

[ebete]

Update the untar module to the latest version as the current one still tries to pull from Anaconda's ToS-covered channels, which is problematic for many users.

PR checklist

• This comment contains a description of changes (with reason).
• Make sure your code lints (nf-core pipelines lint).
• Ensure the test suite passes (nextflow run . -profile test,docker --outdir <OUTDIR>).
• Check for unexpected warnings in debug mode (nextflow run . -profile debug,test,docker --outdir <OUTDIR>).
• CHANGELOG.md is updated.
• Merging template updates 3.3.2 #905 is merged into this PR.

[nf-core-bot]

Warning

Newer version of the nf-core template is available.

Your pipeline is using an old version of the nf-core template: 3.3.1.
Please update your pipeline to the latest version.

For more documentation on how to update your pipeline, please see the nf-core documentation and Synchronisation documentation.

[d4straub]

Hi there, thanks for the PR! The failing CI tests seem to be due to a change in output compared to the expected. Those tests are relatively new and unfortunately there seems to be still a bug because its not telling what file/entry/change is the actual cause. An update to the most recent nf-core template might help, but that isnt in the scope of this PR, ofc.
What changes are taking place could be checked by running in the directory containing the pipeline code:

nf-test test tests/doubleprimers.nf.test --profile=+singularity --update-snapshot --clean-snapshot --verbose
nf-test test tests/reftaxcustom.nf.test --profile=+singularity --update-snapshot --clean-snapshot --verbose

I might be able to do that soon-ish.
Edit: I rather attempt the template update: #905

[ebete]

@d4straub Thanks. The test suite run locally finishes without failure at least. Once 3.3.2 merges I'll try to merge it in here and try again to see if it makes any difference.

[d4straub]

So the template update indeed did its job, the reason for the failing tests is now detailed as expected.
In files tests/doubleprimers.nf.test.snap & tests/reftaxcustom.nf.test.snap the entry "untar": 1.3 has to be replaced by "untar": 1.34. When you will have done that, the tests should pass.
Other than that then changes seem fine.

[d4straub]

Great, thanks!
Is it fine to you when I merge it? Usually I find merging is up to the opener of the PR, but you seem to be no nf-core member and might be not able to hit that merge button?

[ebete]

@d4straub yes please go ahead :). I'm indeed not able to merge as I don't have write access.

Thanks for reviewing

[ebete]

Thank you.
A small thing I just now noticed: I put the PR under the wrong section in the CHANGELOG. It should most likely be moved under the Dependencies section instead of Fixed. Sorry about that

[d4straub]

Hi, no problem, before a release I typically look through the CHANGELOG anyway and modify if needed.