search

jonase / eastwood

Clojure lint tool
1.08k stars 66 forks source link

Allow suppression of any linter via Eastwood config files #433

Closed jeffkayser closed 2 years ago

jeffkayser commented 2 years ago

It would be nice to be able to suppress any linter supported by Eastwood, instead of the subset supported. What's the reasoning behind disallowing this?

For my use case, I can work around it since I'm calling eastwood.lint/lint and just getting back data I can choose to ignore, but it'd be nice to control this using Eastwood's standard configuration mechanism.

vemv commented 2 years ago

I think :linters / :exclude-linters / :add-linters is what you're looking for, as documented in https://github.com/jonase/eastwood/tree/323568a076544b98c2105e7652758ddd791509a7#usage

The subset you refer to is not for supressing linters altogther - it's for declaring fine-grained suppressions based on code patterns. Examples.

Finally, it's worth considering the newer ignored faults mechanism which works on a file/line basis - ideal for occasional exceptions which wouldn't warrant disabling a whole linter.

Issue reports always welcome for false positives/negatives - Eastwood strives to be remarkably accurate, needing little config or suppressing.

Cheers - V