validate_params in the nextflow schema defaults to true:
|
"validate_params": { |
|
"type": "boolean", |
|
"description": "Boolean whether to validate parameters against the schema at runtime", |
|
"default": true, |
|
"fa_icon": "fas fa-check-square", |
|
"hidden": true |
|
}{% if test_config %}, |
However, according to the release review guidelines, boolean parameters should default to false:
https://github.com/nf-core/website/blob/c8a6b02cd2a0d42489f0431313bb9135963977d1/sites/docs/src/content/docs/checklists/reviews/pipeline_release_pr.md?plain=1#L52-L54
For consistency with the guidelines, shouldn't validate_params be renamed skip_param_validation and default to false instead?