Skip to content

Fix false error with task ext blocks in strict syntax#6569

Merged
bentsherman merged 2 commits intomasterfrom
fix-task-ext-block-strict-syntax
Nov 18, 2025
Merged

Fix false error with task ext blocks in strict syntax#6569
bentsherman merged 2 commits intomasterfrom
fix-task-ext-block-strict-syntax

Conversation

@bentsherman
Copy link
Member

Close #4903

This PR fixes an issue with process ext blocks specified in a config profile

The original issue was about ext blocks under different selectors being merged when using the old syntax parser. The strict parser doesn't have this problem, but had some other issues that this PR addresses.

Since the old config parser is very complicated and fragile, and will be deprecated in 26.04, I have opted to fix the issues only in the strict parser.

Signed-off-by: Ben Sherman <bentshermann@gmail.com>
@netlify
Copy link

netlify bot commented Nov 14, 2025

Deploy Preview for nextflow-docs-staging canceled.

Name Link
🔨 Latest commit ce1b6b6
🔍 Latest deploy log https://app.netlify.com/projects/nextflow-docs-staging/deploys/691ca64623cecb0008df7cc1

@bentsherman bentsherman merged commit fe8065a into master Nov 18, 2025
23 checks passed
@bentsherman bentsherman deleted the fix-task-ext-block-strict-syntax branch November 18, 2025 17:37
@pditommaso
Copy link
Member

I've tried to backport to 25.10.1 but got a conflict. skipping for now.

bentsherman added a commit that referenced this pull request Dec 1, 2025
Signed-off-by: Ben Sherman <bentshermann@gmail.com>
bentsherman added a commit that referenced this pull request Dec 1, 2025
Signed-off-by: Ben Sherman <bentshermann@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Unexpected concatenation of processes ext {} map in profiles directive

3 participants