-
Notifications
You must be signed in to change notification settings - Fork 749
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 subworfklow + schema for references #7412
base: master
Are you sure you want to change the base?
Conversation
// JUST TO TEST THE SUBWORKFLOW | ||
|
||
workflow UTILS_REFERENCES { | ||
take: | ||
references | ||
param_file | ||
param_value | ||
attribute_file | ||
attribute_value | ||
basepath | ||
|
||
main: | ||
// GIVING up writing a test for the functions, so writing a subworkflow to test it | ||
references_file = extract_references_file(references, param_file, attribute_file, basepath) | ||
references_value = extract_references_value(references, param_value, attribute_value) | ||
|
||
emit: | ||
references_file | ||
references_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.
// JUST TO TEST THE SUBWORKFLOW | |
workflow UTILS_REFERENCES { | |
take: | |
references | |
param_file | |
param_value | |
attribute_file | |
attribute_value | |
basepath | |
main: | |
// GIVING up writing a test for the functions, so writing a subworkflow to test it | |
references_file = extract_references_file(references, param_file, attribute_file, basepath) | |
references_value = extract_references_value(references, param_value, attribute_value) | |
emit: | |
references_file | |
references_value | |
} |
This is totally useless, but that was the only way I could find to actually test the functions
subworkflows/nf-core/utils_references/tests/main.function.nf.test
Outdated
Show resolved
Hide resolved
- pipeline | ||
- references | ||
components: [] | ||
input: [] |
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.
You need to sort this out, 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.
Actually no.
I just added a workflow because I couldn't properly tests the functions.
The actual workflow is never used, I'm just using the functions and the assets.
Why is this a subworkflow and not a module? |
because it has no process, hence it cannot be a module |
PR checklist
Closes #XXX
versions.yml
file.label
nf-core modules test <MODULE> --profile docker
nf-core modules test <MODULE> --profile singularity
nf-core modules test <MODULE> --profile conda
nf-core subworkflows test <SUBWORKFLOW> --profile docker
nf-core subworkflows test <SUBWORKFLOW> --profile singularity
nf-core subworkflows test <SUBWORKFLOW> --profile conda