-
Notifications
You must be signed in to change notification settings - Fork 155
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
Enable linters for test packages #2962
Comments
/assign |
This should not have been closed in #3052 |
@anishbista60 can you please provide more context to the PRs mentioning that linters are actually being added and not removed? |
@hairyhum ok ok . Sorry , description was wrong. |
Refs: #3034 #2962 -- Signed-off-by: Anish Bista <[email protected]> Co-authored-by: Julio López <[email protected]>
@julio-lopez I think it should not be closed. |
It was automatically closed because this PR's description said "fixes: #2962" @anishbista60 Please make sure that the PRs and the commits say "Ref #2962" and not "fixes". |
Is your feature request related to a problem? Please describe.
As of now linters are disabled from test packages in .golangci.yaml file:
Describe the solution you'd like
After enabling this linter, fix all kind of linter issues coming up in test packages.
The text was updated successfully, but these errors were encountered: