Filipe G. Vieira
Filipe G. Vieira
> @fgvieira I've looked at this again, and as I noted above, this PR will not print the shell cmd on dry run - only actual execution. Is this useful...
@coderabbitai review
It seems to have been fixed!
Thanks for your contribution! It seems all commands are quite similar and even share some options. Would it make sense to have a single wrapper for all subcommands?
Maybe consider using the function `move_files` from `snakemake-wrapper-utils`, like in here: https://github.com/snakemake/snakemake-wrappers/blob/60860c85821fd48a6633d63a9850fb0486802b4a/bio/tmb/pyeffgenomesize/wrapper.py#L53-L54
It seems you are using the old repo on Bitbucket. Can you try using the GitHub repo?
But your snakemake is trying to connect to the bitbucket repo: `Failed to open environment file https://bitbucket.org/snakemake/snakemake-wrappers`. What version of snakemake are you using? And what command?
It seems you are using `snakemake v5.31.1`, no? If so, that version is almost 5 years old! I'd strongly recommend using a more recent version of `snakemake`.
Do you think it would be possible to add a test case?
And it would be great of you could also add it to the `meta.yaml` file. Just so people know what the `param` is supposed to do. :smile: