Skip to content

Nextflow 26 strict syntax fixes and lint CI - #10

Merged
jbrestel merged 2 commits into
mainfrom
nextflow26-strict-syntax
Oct 16, 2026
Merged

jbrestel merged 2 commits into
mainfrom
nextflow26-strict-syntax

Conversation

@jbrestel

Copy link
Copy Markdown
Member

Summary

Syntax-only update for Nextflow 26 strict syntax; no behavior change.

  • Add nextflow lint CI via the shared VEuPathDB/.github reusable workflow.
  • depth_policy: replace @field constants with zero-arg functions; C-style (long) casts -> 'as long'
  • concatenate_fastq: call local closure via asList.call(...)
  • sratools/fasterqdump: move errorStrategy/maxRetries directives above input/output sections
    depth_policy nf-tests (38) pass on both 25.04.0 and 26.10.0.

Test plan

  • CI red on lint-only commit (8 errors, 26 warnings), green with fixes (25 warnings)
  • nextflow lint locally on Nextflow 26.10.0: 0 errors

🤖 Generated with Claude Code

jbrestel and others added 2 commits October 13, 2026 16:24
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
- depth_policy: replace @field constants with zero-arg functions; C-style (long) casts -> 'as long'
- concatenate_fastq: call local closure via asList.call(...)
- sratools/fasterqdump: move errorStrategy/maxRetries directives above input/output sections

depth_policy nf-tests (38) pass on both 25.04.0 and 26.10.0.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@jbrestel
jbrestel merged commit ca3598e into main Oct 16, 2026
2 checks passed
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