-
Notifications
You must be signed in to change notification settings - Fork 492
CICD: Add golangci-lint-action to CI/CD pipeline #3313
Copy link
Copy link
Closed
Labels
github_actionsPull requests that update Github_actions codePull requests that update Github_actions code
Description
Is your feature request related to a problem? Please describe.
It would be nice to enforce the default golangci-lint linters.
Describe the solution you'd like
Add this to the PR pipeline: https://github.com/golangci/golangci-lint-action
- Run with the "default" linters. Not "all".
- Add linter "predeclared" (
but it should skip).integrationTest/integration_test.go- NOTE:
integration_test.gohas been remediated. No need to skip it. --Tom 2025-01-08
- NOTE:
- Add linter "godot" but not until bug (Multi-line comments disappear tetafro/godot#39) is fixed.
- Add these linters. They currently have no findings but might in the future:
- asasalint
- asciicheck
- bidichk
- decorder
- durationcheck
- Investigate these linters. They will require code changes but they seem to have value.
- containedctx
- bodyclose
- copyloopvar
- errname
- errorlint
Describe alternatives you've considered
n/a
Additional context
n/a
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
github_actionsPull requests that update Github_actions codePull requests that update Github_actions code