Skip to content

Static typing fixes - #46

Merged
markquintontulloch merged 3 commits into
mainfrom
static_typing_updates
Oct 12, 2026
Merged

markquintontulloch merged 3 commits into
mainfrom
static_typing_updates

Conversation

@markquintontulloch

Copy link
Copy Markdown
Contributor

Some changes that were missing when switching to static typing

@Dishalodha

Copy link
Copy Markdown
Collaborator

Why do we want to remove the when condition? As per nf-core this gives a bit more flexibility to make changes in the config of a pipeline.

@markquintontulloch

Copy link
Copy Markdown
Contributor Author

Why do we want to remove the when condition? As per nf-core this gives a bit more flexibility to make changes in the config of a pipeline.

From this tutorial:
"Process when section
The process when section is discouraged. As a best practice, conditional logic should be implemented in the calling workflow (e.g. using an if statement or filter operator) instead of the process definition."

@Dishalodha

Copy link
Copy Markdown
Collaborator

thank you @markquintontulloch, For the new version it seems to be depricated. Happy to merge.

Deprecated in version 26.10 Implement conditional logic in the calling workflow (e.g. an if statement or [filter](https://docs.seqera.io/nextflow/reference/operator#filter) operator), or use the [process.when](https://docs.seqera.io/nextflow/config#disabling-processes) config setting to disable a process at runtime.

@markquintontulloch
markquintontulloch merged commit aea51f5 into main Oct 12, 2026
6 of 16 checks passed
@markquintontulloch
markquintontulloch deleted the static_typing_updates branch October 13, 2026 12:34
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