Lift CI actions and golangci-lint - #11
Merged
Merged
Conversation
checkout v4 to v7, setup-go v5 to v7, golangci-lint-action v7 to v9 with the linter pinned at v2.12.2. go-version stays at 1.24 to match the go directive. branches and tags are quoted as ["**"], the empty form works but actionlint flags it. the newer linter reports 11 modernize issues: atomic.Int32/Int64 instead of the free functions in tests, and slices.Backward for the middleware loops.
umputun
approved these changes
Aug 19, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Brings the workflow up to what the rest of the organisation runs:
actions/checkoutv4 to v7,actions/setup-gov5 to v7, andgolangci-lint-actionv7 to v9 with the linter pinned at v2.12.2.go-versionstays at "1.24" to match thegodirective.branches:andtags:are now written as["**"]. The empty form behaves the same way, but actionlint reports it as an empty string.The newer linter reports 11
modernizeissues, fixed here since CI would otherwise be red:atomic.Int32/atomic.Int64in place of the freeatomic.AddInt32/atomic.LoadInt32calls in tests, andslices.Backwardfor the two middleware loops inUse. Both conversions are mechanical and preserve the existing order and semantics.