Skip to content

build: bump Go to 1.27.2, update dependencies and pin go-test-coverage - #18

Merged
christiangda merged 2 commits into
mainfrom
chore/update-go-deps
Oct 13, 2026
Merged

christiangda merged 2 commits into
mainfrom
chore/update-go-deps

Conversation

@christiangda

@christiangda christiangda commented Oct 13, 2026 •

Copy link
Copy Markdown
Contributor

Summary

  • Bump Go from 1.26.0 to 1.27.2 in go.mod. CI reads the version from go-version-file. The README badge and requirements now say Go 1.27+.
  • Update Go dependencies:
    • golang.org/x/sync v0.23.0 → v0.24.0
    • golang.org/x/sys v0.47.0 → v0.49.0 (indirect)
  • Pin go-test-coverage to v2.20.0 in pr.yaml and release.yml. It was @latest.

Why the CI pin

The test coverage step was failing (see #17). The cause was not low coverage. @latest started resolving to v2.20.0, which requires Go ≥ 1.27, while CI was running Go 1.26.0 with GOTOOLCHAIN=local:

go: github.com/vladopajic/go-test-coverage/v2@v2.20.0 requires go >= 1.27 (running go 1.26.0; GOTOOLCHAIN=local)

The Go 1.27.2 bump fixes that. The tool stays pinned so a future release can't break CI the same way. On the last green run, coverage was 85.1%, above the 75% threshold.

Note: anyone using the library now needs Go 1.27+.

This PR replaces #17, which can be closed.

Testing

  • go build ./..., go vet ./... and go test -race ./... (unit tests) pass locally on Go 1.27.2.
  • I couldn't run the Valkey integration tests locally because no container runtime was available. CI runs them against the Valkey service.

🤖 Generated with Claude Code

- golang.org/x/sync v0.23.0 => v0.24.0
- golang.org/x/sys v0.47.0 => v0.49.0 (indirect)
- Pin go-test-coverage to v2.19.0 in CI: v2.20.0 (resolved by @latest)
  requires go >= 1.27, while CI runs go 1.26.0 from go.mod with
  GOTOOLCHAIN=local, which broke the "test coverage" step.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@christiangda christiangda self-assigned this Oct 13, 2026
- go.mod: go 1.26.0 => 1.27.2 (CI picks it up via go-version-file)
- README: update Go version badge and requirements to 1.27+
- Bump pinned go-test-coverage to v2.20.0, now compatible with the
  Go 1.27 toolchain

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@christiangda christiangda changed the title build(deps): update Go dependencies and pin go-test-coverage build: bump Go to 1.27.2, update dependencies and pin go-test-coverage Oct 13, 2026
@christiangda
christiangda merged commit 4ccae57 into main Oct 13, 2026
5 checks passed
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.

1 participant