Simon Pearce
Simon Pearce
@julicudini , are you still working on this?
> I think this was fixed in #6194 already I think the comment chain above is suggesting there are other modules that need adjusting in a similar manner, that weren't...
Um, the module doesn't take the index?
@camlloyd , do you need to update the module to take the index too?
Hi @bounlu . This change needs to be made first in https://github.com/nf-core/modules before it is propagated here. That will involve changing the docker, singularity and conda definitions (the third through...
I think you need to disable `SC2086` too, as most of the output you shared is things like: ``` ${args} \ ^-----^ SC2086 (info): Double quote to prevent globbing and...
Actually, the `$variable` is evaluating as a Nextflow variable, not within the shell context, so several of the checks aren't relevant.
Paging @edmundmiller
Looking at @vagkaratzas 's comment, is there a reason why one of the tests uses a github link rather than the test_data?
> Looking at @vagkaratzas 's comment, is there a reason why one of the tests uses a github link rather than the test_data? Ok, trying the dataset that the others...