Skip to content

Adopt golangci-lint for static analysis #31

@JamieMagee

Description

@JamieMagee

There's no linter in CI. go vet runs implicitly via go test, but that's about it.

I ran golangci-lint with its default linters against the repo and got ~55 findings. Nothing urgent, but these accumulate. A linter in CI catches them at PR time.

Proposal

  1. Add a minimal .golangci.yml using only the default linters.
  2. Fix or //nolint existing findings so main passes clean.
  3. Add a lint Makefile target.
  4. Add a CI job using golangci-lint-action, parallel to the existing test job.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type
    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions