Update all dependencies - #1198
Open
cpuguy83 wants to merge 2 commits into
Open
Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
Updates Dalec’s Go toolchain baseline to Go 1.26 and refreshes Go dependencies.
Changes:
- Raises the module and CI toolchain to Go 1.26.
- Updates BuildKit, containerd, OpenTelemetry, and related dependencies.
- Updates test and mirrored builder images.
Reviewed changes
Copilot reviewed 8 out of 9 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
.github/copilot-instructions.md |
Documents Go 1.26 requirement. |
.github/workflows/ci.yml |
Uses Go 1.26 in CI. |
.github/workflows/retag.yml |
Uses Go 1.26 for retagging. |
.github/workflows/retag/images.yml |
Mirrors the Go 1.26 image. |
go.mod |
Raises Go baseline and dependency versions. |
go.sum |
Refreshes dependency checksums. |
test/fixtures/phony.go |
Builds fixture with Go 1.26. |
test/fixtures/signer.go |
Builds signer with Go 1.26. |
test/git_services/teststate.go |
Builds Git test server with Go 1.26. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Contributor
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 9 out of 10 changed files in this pull request and generated no new comments.
Suppressed comments (2)
go.mod:21
- The BuildKit library is raised to v0.32.2, but the BuildKit image used by CI remains pinned to v0.31.2 in
Dockerfile:5. This leaves a declared dependency unupdated and makes integration tests exercise the new client/frontend code against the previous daemon release. Update the Dockerfile tag and digest to v0.32.2 as part of this upgrade.
github.com/moby/buildkit v0.32.2
go.mod:3
- The Go 1.26 requirement is only applied to the root module; the separate
website/go.modstill declaresgo 1.25.9. This leaves part of the repository on the old toolchain despite the PR's stated repository upgrade and the updated “Go 1.26+” development requirement. Update the website module (and its sums if needed) in the same change.
go 1.26.3
This forces an update to go1.26 Signed-off-by: Brian Goff <cpuguy83@gmail.com>
Signed-off-by: Brian Goff <cpuguy83@gmail.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This forces an update to go1.26