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.

Sign up for GitHub

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

Jump to bottom

cleanup: add linters on error checks and return #989

Closed

mtardy opened this issue May 11, 2023 · 1 comment

Assignees

Labels

Member

mtardy commented May 11, 2023

After the following cleanup:

It would be nice to enable these linters about errors, but it will be some amount of work since they return a lots of alerts.

errcheck (this one is particularly important)
nilerr
errorlint
nilnil

We could see if we can then catch situations where we use the return value before checking the error as well.

mtardy added the kind/cleanup label

mtardy self-assigned this

mtardy mentioned this issue

cleanup: add new linters and fix bugs and alerts #983

Draft

Contributor

jrfastab commented Oct 20, 2023

this is old and happy with linters now. closing...

jrfastab closed this as completed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment