-
Notifications
You must be signed in to change notification settings - Fork 778
Module System Implementation #6768
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
bentsherman
merged 26 commits into
251117-module-system
from
251117-module-system-implementation
Mar 10, 2026
Merged
Changes from 24 commits
Commits
Show all changes
26 commits
Select commit
Hold shift + click to select a range
f5b6b86
Add first implementation of module CLI commands
jorgee f5822a5
add print of process outputs
jorgee 24a741f
rename ModuleSpec and PipelineSpec
jorgee 2432e90
add documentation and tests
jorgee 31bf49c
add registry config as config extension
jorgee b091d73
fix NPE in nextflow CLI help
jorgee a03f630
add module info and expect /api path in the config registry url, upda…
jorgee 36e9085
update default registry to include /api
jorgee d4418ed
Merge branch '251117-module-system' into 251117-module-system-impleme…
jorgee fc327c4
Fix compilation issue [ci skip]
pditommaso c3b02a8
add review comments and fix compilation and tests
jorgee 73e9707
look for checksum in redirected headers
jorgee c154d35
rename checksum header
jorgee e924509
update docs
jorgee 412ba53
fix NPE when no nextflow.config and change preference in registry url…
jorgee 119c632
update speckit files [ci skip]
jorgee ad0bba2
Merge branch '251117-module-system' into 251117-module-system-impleme…
pditommaso 4826533
Merge branch '251117-module-system' into 251117-module-system-impleme…
jorgee c20a408
rename module subcommands
jorgee 90dc444
address review comments
jorgee afddf21
fix test
jorgee 18a9e70
Remote module inclusion (#6815)
jorgee a2adf6e
update docs
bentsherman 292e4f2
change module remove behaviour to avoid unexpected removes
jorgee e96c6ef
Apply suggestions from code review [ci skip]
jorgee 0b83a2c
remove pipeline spec and modules config
jorgee File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We don't really say in the new docs where
--readsand--indexcome from. It'd be good to have a bit more guidance about how people should know what CLI flags to use.Could we get
nextflow module run [name] --helpto list available flags? That'd be really useful!There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's move this to #6650