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

Feature request: allow disabling certain checks #19

Closed
SuperSandro2000 opened this issue Nov 10, 2021 · 4 comments
Closed

Feature request: allow disabling certain checks #19

SuperSandro2000 opened this issue Nov 10, 2021 · 4 comments

Comments

@SuperSandro2000
Copy link

I want to start applying the tool to nixpkgs. For that I would like to disable certain checks or only enable one to keep the changes small and for example only remove extra ().

@nerdypepper
Copy link
Owner

nerdypepper commented Nov 12, 2021

Understood. For now, I'll add a config file to enable/disable lints. In the future, I think I'll add a way to selectively ignore issues with a comment such as:

#statix ignore
("hello")

@SuperSandro2000
Copy link
Author

I think we want to add which issue should be ignored to the comment like shellcheck does.

@nerdypepper
Copy link
Owner

Oh yeah, I intend to support two syntaxes:

# skip all issues on next line
#statix ignore

# skip specific issues on next line
#statix ignore: useless_parens, empty_let_in

@nerdypepper
Copy link
Owner

With the latest release (v0.4.1), you can disable checks with a configuration file. Please see https://github.com/nerdypepper/statix#configuration. Feel free to reopen or create a new issue if there are bugs with the new flag!

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

No branches or pull requests

2 participants