Skip to content

[BACK-4088] Bump go version to 1.25.x.#898

Merged
lostlevels merged 1 commit intomasterfrom
jimmy-go-125
Apr 6, 2026
Merged

[BACK-4088] Bump go version to 1.25.x.#898
lostlevels merged 1 commit intomasterfrom
jimmy-go-125

Conversation

@lostlevels
Copy link
Copy Markdown
Contributor

No description provided.

toddkazakov
toddkazakov previously approved these changes Jan 9, 2026
@lostlevels lostlevels force-pushed the jimmy-go-125 branch 4 times, most recently from 7a9e9e3 to 5e18542 Compare March 17, 2026 17:46
@lostlevels lostlevels changed the title [BACK-4088] Bump go version to 1.25.5. [BACK-4088] Bump go version to 1.25.x. Mar 17, 2026
Copy link
Copy Markdown
Contributor

@darinkrauss darinkrauss left a comment

Choose a reason for hiding this comment

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

LGTM, with one curiosity question.

Comment thread Makefile
go work edit -use=./private/plugin/$(PLUGIN) && \
go work edit -go=`sed -n 's/^go //p' go.mod` && \
go work edit -toolchain=`sed -n 's/^toolchain //p' go.mod` && \
go work edit -toolchain=`grep -q '^toolchain' go.mod && sed -n 's/^toolchain //p' go.mod || sed -n 's/^go /go/p' go.mod` && \
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Just curious - what is this change for?

Copy link
Copy Markdown
Contributor Author

@lostlevels lostlevels Mar 27, 2026

Choose a reason for hiding this comment

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

go mod tidy for me removed the toolchain go... in the go.mod so the go work edit -toolchain= call would have an empty value causing the build to fail so this was added in case someone else's run of go mod tidy adds it back.

@lostlevels lostlevels merged commit 4119b48 into master Apr 6, 2026
3 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