-
Notifications
You must be signed in to change notification settings - Fork 952
Open
Labels
bugSomething isn't workingSomething isn't working
Description
Have you checked the docs?
Description of the bug
The seqkit replace module incorrectly uses the -i flag, I think the intended usage was to specify the input (incorrect). The -i is --ignore-case. Seqkit uses positional argument to specify the input.
It should be removed from here:
modules/modules/nf-core/seqkit/replace/main.nf
Lines 33 to 38 in 04dcc4d
| seqkit \\ | |
| replace \\ | |
| ${args} \\ | |
| --threads ${task.cpus} \\ | |
| -i ${fastx} \\ | |
| -o ${prefix}.${endswith} |
It's not the end of the word, but it may break some peoples code. It should however still be removed as case is important.
Metadata
Metadata
Assignees
Labels
bugSomething isn't workingSomething isn't working