Skip to content

docs: Add longer NXF_SYNTAX_PARSER descriptions#6637

Merged
christopher-hakkaart merged 4 commits intonextflow-io:masterfrom
christopher-hakkaart:docs-syntax-parser
Dec 5, 2025
Merged

docs: Add longer NXF_SYNTAX_PARSER descriptions#6637
christopher-hakkaart merged 4 commits intonextflow-io:masterfrom
christopher-hakkaart:docs-syntax-parser

Conversation

@christopher-hakkaart
Copy link
Collaborator

No description provided.

Signed-off-by: Christopher Hakkaart <chris.hakkaart@seqera.io>
@netlify
Copy link

netlify bot commented Dec 4, 2025

Deploy Preview for nextflow-docs-staging ready!

Name Link
🔨 Latest commit 88bf946
🔍 Latest deploy log https://app.netlify.com/projects/nextflow-docs-staging/deploys/69325b73f08d7c00088c7ebf
😎 Deploy Preview https://deploy-preview-6637--nextflow-docs-staging.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Signed-off-by: Christopher Hakkaart <chris.hakkaart@seqera.io>
Signed-off-by: Christopher Hakkaart <chris.hakkaart@seqera.io>
Signed-off-by: Christopher Hakkaart <chris.hakkaart@seqera.io>
@christopher-hakkaart christopher-hakkaart marked this pull request as ready for review December 5, 2025 13:21
@christopher-hakkaart christopher-hakkaart requested a review from a team as a code owner December 5, 2025 13:21
@christopher-hakkaart christopher-hakkaart merged commit 23c277a into nextflow-io:master Dec 5, 2025
26 checks passed
@christopher-hakkaart christopher-hakkaart deleted the docs-syntax-parser branch December 5, 2025 13:22
Copy link
Member

@bentsherman bentsherman left a comment

Choose a reason for hiding this comment

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

Please address these comments in a new PR


<h3>Workflow params</h3>

:::note
Copy link
Member

Choose a reason for hiding this comment

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

Should be :::{note}

<h3>Workflow params</h3>

:::note
To use workflow params, set the `NXF_SYNTAX_PARSER` environment variable to `v2`:
Copy link
Member

Choose a reason for hiding this comment

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

The wording kinda suggests that this is all you need to do in order to use workflow params, maybe better to phrase it the same way that you did in workflow.md

Copy link
Member

Choose a reason for hiding this comment

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

Also this note needs to be added to "Type annotations" and "New syntax for workflow handlers"

In order to use this feature, you'll need to:
To use this feature:

1. Set the `NXF_SYNTAX_PARSER` environment variable to `v2`:
Copy link
Member

Choose a reason for hiding this comment

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

I would still phrase it as "Enable the strict syntax by setting..." so that there is a link to the strict syntax. Enabling this flag in practice will require other changes to pipeline code which are explained on that page

export NXF_SYNTAX_PARSER=v2
```

Without this environment variable set, Nextflow will not enforce strict syntax rules. Add this to your shell profile (e.g., `~/.bashrc`, `~/.zshrc`) or set it before you run Nextflow commands.
Copy link
Member

Choose a reason for hiding this comment

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

I'm not sure we should recommend setting this flag globally, although it may be sensible for some. I suspect most people will need to toggle it at a pipeline-level for the time being.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants