Max Cummins
05/07/2026, 3:12 PMnf-core tools subworkflow install will install all module dependencies using meta.yml
precise commits can be fed to nf-core tools module</subworkflow> install , but (i believe) there is no option for adding a precise commit to a meta.yml as follows, such that its dependencies are also pinned (although perhaps the commit state of modules are pinned to the parent subworkflow via the command line install when a precise commit is provided?):
components:
- samtools/sort:
git_remote: "<https://github.com/nf-core/modules.git>"
branch: master
git_sha: "abc123..."
In practice, this means that as nf-core modules change, a subworkflow can be installed on two different dates - the subworkflow code may be identical but the modules that nf-core tools installs as dependencies can differ. While I appreciate that changes to a module in nf-core trigger test cascades for subworkflows which use said modules, this doesn't really handle the underlying issue.
This version control issue has been a bit of a headache for my team for some time and I thought it was time to bring it up here and potentially look into making a PR.
This would probably touch nf-core lint as well.
Anyone have any thoughts on this? would appreciate it.Rike Hanssen
05/07/2026, 5:02 PMMax Cummins
05/07/2026, 9:24 PMMax Cummins
05/07/2026, 9:25 PMMax Cummins
05/07/2026, 9:26 PMArthur
05/08/2026, 1:50 AMpip) adding version pinning to individual subworkflow meta yml is a bad idea because what do you do when two subworkflows pin different - potentially incompatible - versions?
Consider additionally "patched" modules. I know lots of people are a fan but if you have a very common module (something like samtools) that you patch for some use case in your pipeline this could easily affect the way a subworkflow running that module operates in your pipeline.
TL;DR: the problem is the way subworkflows are designed currently. Within the current design I don't think there is a good or safe solution to pinning dependency versions.Arthur
05/08/2026, 1:53 AMPhil Ewels
05/08/2026, 4:32 AMPhil Ewels
05/08/2026, 4:34 AMMahesh Binzer-Panchal
05/08/2026, 7:16 AMArthur
05/08/2026, 7:31 AM