Skip to content

Update to Go 1.27 and refresh dependencies - #178

Merged
wimaha merged 1 commit into
mainfrom
deps/go-1.27
Oct 19, 2026
Merged

wimaha merged 1 commit into
mainfrom
deps/go-1.27

Conversation

@wimaha

@wimaha wimaha commented Oct 19, 2026

Copy link
Copy Markdown
Owner

What

  • go.mod: go 1.27.0. CI (default, dev, release): GO_VERSION: ^1.27.
  • Dockerfile: golang:1.27.2, the latest patch release (was 1.27.1).
  • Dependencies refreshed with go get -u ./..., limited to modules that are actually built: charmbracelet/x/*, colorprofile, go-colorful, go-colorable, go-isatty, go-runewidth, logrus, terminfo, x/exp, x/sys.
  • Forks unchanged: wimaha/vehicle-command v0.0.8 and wimaha/ble_BleConnectFix (b933489) stay pinned by their replace directives. go get -u raised the vehicle-command require line to the upstream v0.4.1. I set it back to v0.2.1, because the replace ignores it anyway and the change would suggest an upgrade that does not happen.
  • CHANGELOG: building from source needs Go 1.27.

Testing

  • go vet ./..., golangci-lint run (0 issues), go test -race ./...
  • CGO_ENABLED=0 builds for linux/amd64, arm64, arm/v6 and arm/v7, the platforms of the Docker images.
  • Local binary started: log output is formatted as before, which matters because the log library's dependencies moved by several versions. /api/proxy/1/status answers 200.
  • The :pr-<number> image from this PR can be used for a check on the Pi.

🤖 Generated with Claude Code

- go.mod: go 1.27.0; CI (default, dev, release): GO_VERSION ^1.27.
- Dockerfile: golang:1.27.2 (latest patch release).
- go get -u ./... for the modules that are built: charmbracelet/x/*,
  colorprofile, go-colorful, go-colorable, go-isatty, go-runewidth, logrus,
  terminfo, x/exp, x/sys. The vehicle-command and go-ble forks stay pinned
  by their replace directives; the vehicle-command require line stays at
  v0.2.1 because the replace ignores it anyway.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@github-actions

Copy link
Copy Markdown

✅ Ready to merge

This only raises the Go version and updates logging and terminal libraries. The BLE code, the car library fork and the HTTP API are untouched.

Impact on existing users: None. Behaviour should be the same. The only visible risk is a small change in log formatting.

To-do for the author

  • Nothing left to do

Test before merging: Start the :pr-178 image on a Pi (ideally arm/v6 or v7). Check that the logs look normal and that /api/proxy/1/status answers.

Technical details
  • go.mod: vehicle-command and ble_BleConnectFix are still pinned by replace, so BLE behaviour does not change.
  • Dockerfile: only the builder image changes (golang:1.27.2). The runtime image and platforms are unchanged.
  • Dependencies: logrus 1.9.3 → 1.10.2 and charmbracelet/* moved by several versions. Logging only. Not verified: whether log output is still the same on a real device. I read the diff and did not build or run it.
  • go.sum: some old entries were removed and go.yaml.in/yaml/v3 was added. This is normal for go mod tidy.

@wimaha
wimaha merged commit 5a0ea25 into main Oct 19, 2026
10 checks passed
@wimaha
wimaha deleted the deps/go-1.27 branch October 19, 2026 13:32
@wimaha wimaha mentioned this pull request Oct 19, 2026
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