Skip to content

CICD: Add golangci-lint-action to CI/CD pipeline #3313

@tlimoncelli

Description

@tlimoncelli

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.go has been remediated. No need to skip it. --Tom 2025-01-08
  • 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

Metadata

Metadata

Assignees

Labels

github_actionsPull requests that update Github_actions code

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions