-
Notifications
You must be signed in to change notification settings - Fork 417
Upgrade to Go 1.25 #9771
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Upgrade to Go 1.25 #9771
Conversation
nopcoder
commented
Dec 9, 2025
- Update dockerfile
- Update go.mod
- Update github workflows
- Update esti
- Update jpebble test docker container
- Github workflow match go version based on go.mod
- Update dockerfile - Update go.mod - Update github workflows - Update esti - Update jpebble test docker container - Github workflow match go version based on go.mod
There was a problem hiding this 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 project from Go 1.24.4 to Go 1.25.5, but there is a critical issue: Go 1.25.5 does not exist. As of January 2025, the latest stable Go release was 1.23.x, and Go 1.24 had not yet been officially released.
Key changes attempted:
- Update Go version from 1.24.4 to 1.25.5 across all go.mod files
- Upgrade GitHub Actions setup-go from v4 to v6 with go-version-file for dynamic version management
- Update Docker images from golang:1.24-alpine to golang:1.25-alpine
- Update various Go dependencies to versions compatible with the intended Go upgrade
Reviewed changes
Copilot reviewed 22 out of 23 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| go.mod | Update Go version to 1.25.5 and upgrade dependencies (nonexistent version) |
| go.work | Update workspace Go version to 1.25.5 (nonexistent version) |
| webui/go.mod | Update Go version to 1.25.5 (nonexistent version) |
| modules/*/factory/go.mod | Update Go version to 1.25.5 across all module factory files (nonexistent version) |
| go.sum, go.work.sum | Update dependency checksums for new Go version and dependencies |
| Dockerfile | Update base image to golang:1.25-alpine (nonexistent image tag) |
| esti/ops/docker-compose-common.yaml | Update esti container to golang:1.25-alpine (nonexistent image tag) |
| clients/spark/src/test/scala/io/treeverse/jpebble/BlockParserSpec.scala | Update test container to golang:1.25-alpine (nonexistent image tag) |
| .github/workflows/*.yaml | Upgrade actions/setup-go from v4 to v6 and switch from hardcoded version to go-version-file for better maintainability |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
arielshaqed
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Neat, thanks!
But worst Copilot review EVER.