-
Notifications
You must be signed in to change notification settings - Fork 545
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
chore: add more linters and make changes #3256
Conversation
@tbruyelle @jeronimoalbi do you guys like these added linters? No need to add them if they are more trouble. I tried to add ones that weren't too opinionated and only would enforce some best practices. |
I tried to have a look at all the changes, and that seems fine to me, the ones you enabled look definitively useful. |
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## main #3256 +/- ##
=======================================
Coverage 20.07% 20.08%
=======================================
Files 385 385
Lines 30797 30829 +32
=======================================
+ Hits 6184 6192 +8
- Misses 24024 24049 +25
+ Partials 589 588 -1
|
* errjsonchk * dupword * errorlint * exhaustive * godot * nolintlint * reassign * test helper * finalize linter file * remove unwrap * fix typo * create TODO for markdownviewr * lint fixes * re-add unwrap
Adding:
decorder
dupword
errchkjson
errorlint
exhaustive
godot
grouper
nolintlint
reassign
tenv
thelper