Side quest for an old tiny issue <github.com/nf-co...
# sarek_dev
r
Side quest for an old tiny issue github.com/nf-core/sarek/pull/2351/changes
👀 1
adressed the reviews @Maxime Garcia
m
just one last question before merging
so by default, it's false
Why not having
mosdepth_fast_mode
instead, and by default true?
r
yep by default nothing should change.
m
I think
mosdepth_fast_mode
is easier to understand than
mosdepth_no_fast_mode
r
ah sure. doesnt' matter to me really. I always find the
--boolean_param false
a bit weird, instead of
--boolean_param
from a usability perspective. but agreee that the param name is easier to understand
there you go
m
Usage docs?
Sorry, just thought about that
Feel free to merge anytime
r
hm I removed them, because it is all in the flag description. I am not sure what the usage doc would add
✅ 1