Skip to content

chore: bump golangci-lint and apply formatting#393

Merged
retr0h merged 1 commit intomainfrom
chore/golangci-lint-bump-and-formatting
May 9, 2026
Merged

chore: bump golangci-lint and apply formatting#393
retr0h merged 1 commit intomainfrom
chore/golangci-lint-bump-and-formatting

Conversation

@retr0h
Copy link
Copy Markdown
Collaborator

@retr0h retr0h commented May 9, 2026

Summary

  • Bump golangci-lint v2.11.4 → v2.12.2 (via go get -tool) and tidy go.sum
  • Bump charmbracelet/x/cellbuf to v0.0.15 to fix an ansi API mismatch (old cellbuf was calling Italic()/Strikethrough()/etc. with no args, but the newer ansi v0.11.7 pulled in by golangci-lint and charm.land/lipgloss/v2 requires a bool argument)
  • Replace reflect.Ptr with reflect.Pointer in internal/validation/validation.go (newer govet flags the legacy alias)
  • Re-run gofumpt + golines across the tree

Test plan

  • `just go::vet` — 0 issues
  • `just go::unit` — all packages pass
  • CI green

🤖 Generated with Claude Code

Bumps golangci-lint and transitive deps, including charmbracelet/x/cellbuf
to v0.0.15 to resolve an ansi API mismatch with the newer
charmbracelet/x/ansi v0.11.7 pulled in transitively.

Replaces reflect.Ptr with reflect.Pointer in validation.AtLeastOneField
to satisfy the newer govet inline check.

Re-runs gofumpt + golines formatting across the tree.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
@retr0h retr0h merged commit 7174a11 into main May 9, 2026
9 checks passed
@retr0h retr0h deleted the chore/golangci-lint-bump-and-formatting branch May 9, 2026 06:08
@codecov
Copy link
Copy Markdown

codecov Bot commented May 9, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

Impacted file tree graph

@@           Coverage Diff            @@
##             main     #393    +/-   ##
========================================
  Coverage   99.91%   99.91%            
========================================
  Files         484      484            
  Lines       24721    24935   +214     
========================================
+ Hits        24699    24913   +214     
  Misses         14       14            
  Partials        8        8            
Files with missing lines Coverage Δ
internal/agent/handler.go 100.00% <100.00%> (ø)
internal/agent/preflight.go 100.00% <100.00%> (ø)
internal/agent/processor.go 100.00% <100.00%> (ø)
internal/agent/processor_certificate.go 100.00% <100.00%> (ø)
internal/agent/processor_interface.go 100.00% <100.00%> (ø)
internal/agent/processor_network.go 100.00% <100.00%> (ø)
internal/agent/processor_ntp.go 100.00% <100.00%> (ø)
internal/agent/processor_route.go 100.00% <100.00%> (ø)
internal/agent/processor_schedule.go 100.00% <100.00%> (ø)
internal/agent/processor_service.go 100.00% <100.00%> (ø)
... and 120 more

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update bf8a191...32131d0. Read the comment docs.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant