Skip to content

Add golangci-lint configuration and CI workflow#3

Merged
lludlow merged 2 commits intomainfrom
claude/add-linting-Bb4zl
Apr 13, 2026
Merged

Add golangci-lint configuration and CI workflow#3
lludlow merged 2 commits intomainfrom
claude/add-linting-Bb4zl

Conversation

@lludlow
Copy link
Copy Markdown
Owner

@lludlow lludlow commented Apr 13, 2026

Set up golangci-lint v2 with the standard linter set (errcheck, govet,
ineffassign, staticcheck, unused) plus gocritic, misspell, unconvert,
and goimports. Add a GitHub Actions workflow that runs the linter on
PRs and pushes to main. Fix all existing lint issues: remove unused
valueString function, correct goimports formatting, and exclude
deferred Close() calls from errcheck.

https://claude.ai/code/session_01WQDT6GkU9KZX57VdE5jZBi

claude added 2 commits April 13, 2026 02:05
Set up golangci-lint v2 with the standard linter set (errcheck, govet,
ineffassign, staticcheck, unused) plus gocritic, misspell, unconvert,
and goimports. Add a GitHub Actions workflow that runs the linter on
PRs and pushes to main. Fix all existing lint issues: remove unused
valueString function, correct goimports formatting, and exclude
deferred Close() calls from errcheck.

https://claude.ai/code/session_01WQDT6GkU9KZX57VdE5jZBi
Replace the "no linter is configured" note with instructions to run
golangci-lint before submitting.

https://claude.ai/code/session_01WQDT6GkU9KZX57VdE5jZBi
@lludlow lludlow merged commit 643b609 into main Apr 13, 2026
1 check passed
@lludlow lludlow deleted the claude/add-linting-Bb4zl branch April 13, 2026 02:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants