Skip to content
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

Open
saima-s opened this issue Jun 26, 2024 · 1 comment
Open

Enable linters for test packages #2962

saima-s opened this issue Jun 26, 2024 · 1 comment

Comments

@saima-s
Copy link
Contributor

saima-s commented Jun 26, 2024

Is your feature request related to a problem? Please describe.
As of now linters are disabled from test packages in .golangci.yaml file:

issues:
  exclude-rules:
    - path: '_test.go'
      linters:
        - errcheck                      # Errors may be ignored in tests.
        - unparam                       # Tests might have unused function parameters.
        - lll
        - dupl
        - misspell
        - nestif
        - gci

Describe the solution you'd like
After enabling this linter, fix all kind of linter issues coming up in test packages.

Copy link

Thanks for opening this issue 👍. The team will review it shortly.

If this is a bug report, make sure to include clear instructions how on to reproduce the problem with minimal reproducible examples, where possible. If this is a security report, please review our security policy as outlined in SECURITY.md.

If you haven't already, please take a moment to review our project's Code of Conduct document.

@infraq infraq added this to To Be Triaged in Kanister Jun 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Kanister
To Be Triaged
Development

No branches or pull requests

1 participant