Sitelet https://github.com/Ensembl/nextflow_modules/pull/46
Skip to content

Static typing fixes - #46

Open
markquintontulloch wants to merge 3 commits into
mainfrom
static_typing_updates
Open

markquintontulloch wants to merge 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."

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