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

Lint test - check all params in nextflow.config are mentioned in main.nf #1038

Closed
ewels opened this issue Apr 26, 2021 · 0 comments
Closed
Labels

Comments

@ewels
Copy link
Member

ewels commented Apr 26, 2021

Sometimes parameters are added in nextflow.config and then removed from the pipeline. These dangling parameters confuse matters as they need to be in the schema and clutter things up. However, they are quite difficult to spot when reviewing.

It would be nice to have a simple test that checks that all params are found in main.nf. May need one or two exceptions, but that's fine.

@ewels ewels added the linting label Apr 26, 2021
ewels added a commit to ewels/nf-core-tools that referenced this issue Apr 26, 2021
@ewels ewels mentioned this issue Apr 26, 2021
4 tasks
@ewels ewels closed this as completed May 10, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant