Skip to content

Upgrade to go1.25.9#1668

Merged
meiji163 merged 3 commits intomasterfrom
meiji163/go1.25.9
May 1, 2026
Merged

Upgrade to go1.25.9#1668
meiji163 merged 3 commits intomasterfrom
meiji163/go1.25.9

Conversation

@meiji163
Copy link
Copy Markdown
Contributor

@meiji163 meiji163 commented Apr 30, 2026

Upgrade go version 1.23.0 -> 1.25.9.

Also fix Dockerfile.test to run unit tests (no integration tests)

Copilot AI review requested due to automatic review settings April 30, 2026 22:28
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR aims to upgrade the repository’s Go toolchain from 1.23.0 to 1.25.9 across local tooling and containerized workflows.

Changes:

  • Bumps the preferred Go version and associated checksums in script/ensure-go-installed.
  • Updates test/build scripts and Docker test invocation to support go test flags (notably -short).
  • Skips several test suites when running in testing.Short() mode.
Show a summary per file
File Description
script/test Runs gofmt with consistent args, scopes diff check to go/, forwards CLI args to go test.
script/ensure-go-installed Updates preferred Go version/checksums; adjusts macOS install extraction step.
go/logic/streamer_test.go Skips suite in short mode.
go/logic/migrator_test.go Skips suite in short mode.
go/logic/applier_test.go Skips suite in short mode.
Dockerfile.test Updates base Go image and runs tests in short mode by default.
Dockerfile.packaging Updates base Go image for packaging builds.

Copilot's findings

Tip

Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

  • Files reviewed: 7/7 changed files
  • Comments generated: 5

Comment thread script/ensure-go-installed Outdated
Comment thread script/ensure-go-installed
Comment thread Dockerfile.test Outdated
Comment thread Dockerfile.packaging Outdated
Comment thread script/test
@meiji163 meiji163 mentioned this pull request May 1, 2026
2 tasks
@meiji163 meiji163 merged commit 4d38923 into master May 1, 2026
9 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.

3 participants