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

Support version topic channels #3093

Open
wants to merge 4 commits into
base: dev
Choose a base branch
from
Open

Conversation

edmundmiller
Copy link
Contributor

@edmundmiller edmundmiller commented Aug 1, 2024

Adds from nf-core/rnaseq#1115, nf-core/rnaseq#1109 and nf-core/nascent#150

This should support legacy version.ymls, version.ymls with the topic version_yaml and eval topics in a versions channel.

PR checklist

  • This comment contains a description of changes (with reason)
  • CHANGELOG.md is updated
  • If you've fixed a bug or added code that should be tested, add tests!
  • Documentation in docs is updated

Otherwise the Workflow is the only Yaml with 4 spaces instead of 2
@@ -258,7 +258,7 @@ manifest {
homePage = 'https://github.com/{{ name }}'
description = """{{ description }}"""
mainScript = 'main.nf'
nextflowVersion = '!>=23.04.0'
nextflowVersion = '!>=23.11.0-edge'
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why not upgrade to the next stable release?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I figured we would but this was the first release to have topic channels.

Comment on lines +131 to +133
|Workflow:
| $workflow.manifest.name: ${getWorkflowVersion()}
| Nextflow: $workflow.nextflow.version
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

what does this syntax do here?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It strips the white space from the multi line string until the delimiter which by default is |

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ah nice :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants