You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Describe the bug
Currently we check our code with golangci-lint, which in itself catches most of the things, but having comments over exported function is not caught by it, it'll be nice to enable both the linters. Currently there are some warnings by this linter on the main branch
To Reproduce
Run: golint ./... | grep -v zz_generated #to ignore the auto-generated files
Expected behavior
No warnings
The existing warning should be removed and the lint should also be part of CI so we don't have any warnings from future
The text was updated successfully, but these errors were encountered:
Describe the bug
Currently we check our code with golangci-lint, which in itself catches most of the things, but having comments over exported function is not caught by it, it'll be nice to enable both the linters. Currently there are some warnings by this linter on the main branch
To Reproduce
Run:
golint ./... | grep -v zz_generated
#to ignore the auto-generated filesExpected behavior
No warnings
The existing warning should be removed and the lint should also be part of CI so we don't have any warnings from future
The text was updated successfully, but these errors were encountered: