-
Notifications
You must be signed in to change notification settings - Fork 635
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
Add process programmatic configuration #4375
Conversation
Signed-off-by: Ben Sherman <[email protected]>
Signed-off-by: Ben Sherman <[email protected]>
✅ Deploy Preview for nextflow-docs-staging ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
@@ -183,6 +183,26 @@ workflow { | |||
Optional params for a process input/output are always prefixed with a comma, except for `stdout`. Because `stdout` does not have an associated name or value like other types, the first param should not be prefixed. | |||
::: | |||
|
|||
### Process config |
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.
Ideally would like to merge #3793 first, then this would become part of the "Workflows" page which is more prominent.
modules/nextflow/src/main/groovy/nextflow/script/ProcessDef.groovy
Outdated
Show resolved
Hide resolved
Well done! Look @drpatelh! ☝️ |
Signed-off-by: Ben Sherman <[email protected]>
Signed-off-by: Ben Sherman <[email protected]>
Signed-off-by: Ben Sherman <[email protected]>
I added the config setting for publishDir defaults as defined in #4186 so that I can further improve the nf-core/rnaseq POC. Doesn't have to be merged with the final PR but it's a simple change. |
This is awesome! Thank you! Will take a closer look when I get a moment. Also, thanks for testing with nf-core/rnaseq! |
Closing in favor of #4510 |
Close #4322
Example test case: