Skip to content

chore: go fix ./...#283

Merged
umputun merged 1 commit into
masterfrom
chore/go-fix
May 8, 2026
Merged

chore: go fix ./...#283
umputun merged 1 commit into
masterfrom
chore/go-fix

Conversation

@paskal
Copy link
Copy Markdown
Collaborator

@paskal paskal commented May 8, 2026

Summary

Apply Go modernization fixes from go fix ./... across both v1 and v2.

Mainly interface{}any (the Go 1.18+ alias) plus small idiomatic updates the tool emits. Found while preparing PRs #279/#280/#281/#282 — kept separate so those security PRs stay focused.

Change

  • 36 files updated, ~84 insertions / ~86 deletions
  • No behaviour changes; pure hygiene
  • Go 1.25+ as required by CLAUDE.md and go.mod

Test

go test -race ./... green in both v1 and v2.

Apply Go modernization fixes from `go fix ./...` across both v1 and v2:

* `interface{}` -> `any` (Go 1.18+ alias)
* Other small idiomatic updates

No behaviour changes; this is a hygiene pass to make the build green
under stricter linters and bring the codebase up to modern Go style.
Run with Go 1.25+ as required by CLAUDE.md and go.mod.
@paskal paskal requested a review from umputun as a code owner May 8, 2026 18:21
@coveralls
Copy link
Copy Markdown

coveralls commented May 8, 2026

Coverage Report for CI Build 25572156724

Coverage remained the same at 84.247%

Details

  • Coverage remained the same as the base build.
  • Patch coverage: 1 uncovered change across 1 file (12 of 13 lines covered, 92.31%).
  • No coverage regressions found.

Uncovered Changes

File Changed Covered %
token/user.go 3 2 66.67%

Coverage Regressions

No coverage regressions found.


Coverage Stats

Coverage Status
Relevant Lines: 3212
Covered Lines: 2706
Line Coverage: 84.25%
Coverage Strength: 7.31 hits per line

💛 - Coveralls

Copy link
Copy Markdown
Member

@umputun umputun left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm, thx

@umputun umputun merged commit c91da02 into master May 8, 2026
9 checks passed
@umputun umputun deleted the chore/go-fix branch May 8, 2026 22:40
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.

3 participants