Include config modules.config in the wrong place ?

[LouisLeNezet]

Description of the bug

Hi !

With the new version of the template the import of the modules.config files as been moved to the bottom of the nextflow.config file.
However all the test config files are imported above in the nextflow.config and therefore any modification to a process present in both a test.config and in the modules.config would be overwritten by the later.

This does happen in phaseimpute and we just moved it next to base.config.

// Load base.config by default for all pipelines
includeConfig 'conf/base.config'

// Load modules.config for DSL2 module specific options
includeConfig 'conf/modules.config'

What were the rational for this modification ?

Command used and terminal output

System information

No response

[mirpedrol]

Hi @LouisLeNezet, the modules.config file has been included the last for a long time (years). It's not common to modify the modules arguments through the test configs, people usually use params for this. But it's true that it is a possibility, so we should change the location where we include this config, thanks for reporting 🙂

[LouisLeNezet]

You're welcome 😉
The rational for me was that in a test case the files are really small, and the parameters for some of the process might need to be changed to be able to run. In our case it was mostly the window size for the chunking of the chromosomes.
But I stumble across this issue when reviewing another PR and it's a bit tricky to see it as the order of import isn't quite visible.