Skip to content
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

Clarify that you must use publish: to produce output files with new syntax #5273

Closed
adamrtalbot opened this issue Sep 2, 2024 · 1 comment · Fixed by #5185
Closed

Clarify that you must use publish: to produce output files with new syntax #5273

adamrtalbot opened this issue Sep 2, 2024 · 1 comment · Fixed by #5185
Labels

Comments

@adamrtalbot
Copy link
Collaborator

The new workflow outputs syntax just suggests you can use an output block to create outputs:

workflow {
    foo(bar())
}

output {
    directory 'results'
}

image

However, this isn't the case and you must use the publish: directive in the process or workflow syntax to make it work:

workflow {
    foo(bar())

    publish:
    foo.out >> 'foo'
}

output {
    directory 'results'
}

We should clarify the documentation to say this.

ewels added a commit that referenced this issue Sep 2, 2024
Closes #5273

[ci-skip]

Signed-off-by: Phil Ewels <phil.ewels@seqera.io>
@ewels
Copy link
Member

ewels commented Sep 4, 2024

Will be fixed by #5185

@ewels ewels linked a pull request Sep 4, 2024 that will close this issue
10 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
3 participants