Skip to content

fix(ci): update golangci-lint action and vacuum install#103

Merged
ericfitz merged 3 commits intomainfrom
fix/ci-workflow-issues
Jan 21, 2026
Merged

fix(ci): update golangci-lint action and vacuum install#103
ericfitz merged 3 commits intomainfrom
fix/ci-workflow-issues

Conversation

@ericfitz
Copy link
Owner

Summary

  • Upgrade golangci-lint-action from v6 to v7 with golangci-lint v2.8.0
  • Fix Go version mismatch error: golangci-lint built with Go 1.24 vs project's Go 1.25.6
  • Fix vacuum installation: script already installs to /usr/local/bin (no need to move)

Root Causes Fixed

  1. Lint & Static Analysis failure: golangci-lint action v6 was using a version built with Go 1.24, incompatible with Go 1.25.6

  2. OpenAPI Validation failure: vacuum install script already places binary in /usr/local/bin, but workflow tried to mv vacuum from current directory

Test plan

  • Verify Lint & Static Analysis job passes
  • Verify OpenAPI Validation job passes
  • Verify Build Verification job passes
  • Verify Unit Tests job passes

🤖 Generated with Claude Code

- Upgrade golangci-lint-action from v6 to v7 with v2.8.0
- Fix Go version mismatch (golangci-lint built with Go 1.24 vs Go 1.25.6)
- Fix vacuum installation (script installs directly to /usr/local/bin)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@dosubot dosubot bot added the bug Something isn't working label Jan 21, 2026
ericfitz and others added 2 commits January 21, 2026 18:42
- Add vacuum-ruleset.yaml to gitignore whitelist (was missing)
- Move exclude-rules from issues to linters.exclusions.rules for v2

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
GORM's Dialector is a named field, not embedded. The db.Dialector.Name()
call is the correct API usage per GORM documentation.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@ericfitz ericfitz merged commit bd62a5d into main Jan 21, 2026
7 checks passed
@ericfitz ericfitz deleted the fix/ci-workflow-issues branch January 22, 2026 01:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant