We’ve had to force merged <https://github.com/nf-c...
# team-maintainers
m
We’ve had to force merged https://github.com/nf-core/modules/pull/11771, because linting failed on an existing
ext.suffix
Have we reached a consensus on how to fix this?
shakefist 2
m
Fix what exactly? The
ext.suffix
should be replaced as an input parameter
m
That was our consensus indeed
m
And that’s the answer to my question 😄
m
Sorry I force merged that PR, I didn't want it to be a bigger time drain that it was already
m
Do we want the
extension
param to be separate or in the input tuple?
n
I'd say add it to the input tuple since that will become the default anyways with record types
i
There is an incoming specification for this, https://github.com/nf-core/website/pull/4210