Skip to content

Conversation

@nopcoder
Copy link
Contributor

@nopcoder 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
@nopcoder nopcoder self-assigned this Dec 9, 2025
Copilot AI review requested due to automatic review settings December 9, 2025 20:24
@github-actions github-actions bot added area/testing Improvements or additions to tests area/UI Improvements or additions to UI dependencies Pull requests that update a dependency file infrastructure build, deploy and release processes area/client/spark area/ci labels Dec 9, 2025
Copy link
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 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.

Copy link
Contributor

@arielshaqed arielshaqed left a 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.

@nopcoder nopcoder added the exclude-changelog PR description should not be included in next release changelog label Dec 16, 2025
@nopcoder nopcoder added the minor-change Used for PRs that don't require issue attached label Dec 20, 2025
@nopcoder nopcoder merged commit bd7e43e into master Dec 20, 2025
68 of 69 checks passed
@nopcoder nopcoder deleted the chore/go-1.25 branch December 20, 2025 16:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/ci area/client/spark area/testing Improvements or additions to tests area/UI Improvements or additions to UI dependencies Pull requests that update a dependency file exclude-changelog PR description should not be included in next release changelog infrastructure build, deploy and release processes minor-change Used for PRs that don't require issue attached

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants