-
Notifications
You must be signed in to change notification settings - Fork 624
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
Workflow output definition (second preview) #5185
base: master
Are you sure you want to change the base?
Conversation
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. |
Signed-off-by: Ben Sherman <[email protected]>
Signed-off-by: Ben Sherman <[email protected]>
Signed-off-by: Ben Sherman <[email protected]>
Signed-off-by: Ben Sherman <[email protected]>
Signed-off-by: Ben Sherman <[email protected]>
Signed-off-by: Ben Sherman <[email protected]>
Signed-off-by: Ben Sherman <[email protected]>
Signed-off-by: Ben Sherman <[email protected]>
Signed-off-by: Ben Sherman <[email protected]>
Signed-off-by: Ben Sherman <[email protected]>
Second preview is ready for review, POCs have also been updated: |
Signed-off-by: Ben Sherman <[email protected]>
Signed-off-by: Ben Sherman <[email protected]>
output { | ||
'fastq' { | ||
path { meta, fastq_1, fastq_2 -> | ||
{ file -> "fastq/${meta.id}/${file.baseName}" } |
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.
<3
Signed-off-by: Ben Sherman <[email protected]>
I just tried the dynamic publishing with some dummy code and it worked perfectly! I'm looking forward to implementing this in my pipelines 🥳 |
void contentType(String value) { | ||
setDefault('contentType', value) | ||
} | ||
|
||
void contentType(boolean value) { | ||
setDefault('contentType', value) | ||
} | ||
|
||
void ignoreErrors(boolean value) { | ||
setDefault('ignoreErrors', value) | ||
} | ||
|
||
void mode(String value) { | ||
setDefault('mode', value) | ||
} | ||
|
||
void overwrite(boolean value) { | ||
setDefault('overwrite', value) | ||
} | ||
|
||
void overwrite(String value) { | ||
setDefault('overwrite', value) | ||
} | ||
|
||
void storageClass(String value) { | ||
setDefault('storageClass', value) | ||
} | ||
|
||
void tags(Map value) { | ||
setDefault('tags', value) | ||
} | ||
|
||
void enabled( boolean value ) { | ||
setDefault('enabled', value) | ||
} |
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.
Is it still possible to control these at target level?
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.
yes
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.
nice, and output top level? only in the config or config and script definition?
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.
only the config. default output directory is results
Close #5103
TODO:
language improvements:
runtime improvements:
new features (might be handled separately):