Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

lower/upper case typos are not reported by parameter validation #1087

Closed
KevinMenden opened this issue May 18, 2021 · 1 comment
Closed

lower/upper case typos are not reported by parameter validation #1087

KevinMenden opened this issue May 18, 2021 · 1 comment
Assignees
Labels
bug Something isn't working

Comments

@KevinMenden
Copy link
Contributor

Due to the code we added to deal with camelCase/camel-case issues, parameters that are mistyped with wrong upper/lower case characters are not reported as unexpected. They are also not used by the pipeline though, so very easy to miss, when not checking the parameter summary.

Example from nf-core/smrnaseq:

When using --clip_R1 instead of --clip_r1, no warning is issued but the --clip_r1 parameter is not overwritten.

@KevinMenden KevinMenden added the bug Something isn't working label May 18, 2021
@KevinMenden KevinMenden self-assigned this May 18, 2021
@KevinMenden
Copy link
Contributor Author

closed by #1088

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant