-
Notifications
You must be signed in to change notification settings - Fork 1.4k
🌱 Bump go to v1.23.7 #11981
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
🌱 Bump go to v1.23.7 #11981
Conversation
@@ -4,7 +4,7 @@ | |||
publish = "docs/book/book" | |||
|
|||
[build.environment] | |||
GO_VERSION = "1.23.6" | |||
GO_VERSION = "1.23.7" |
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.
Doesn't really matter either way but I think we don't really have to keep netlify on the exact same patch version. It's just building our book
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.
Probably doesn't matter, but I think it's safer to be in the habit of bumping it along with everything else. Otherwise, we'll forget and it will lag behind (which again, probably doesn't matter much...until it does).
/lgtm |
LGTM label has been added. Git tree hash: 8e40725d6a4918d2ee0b281a9d901e9bab0837f7
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: sbueringer The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
What this PR does / why we need it:
Updates the Go toolchain to the latest patch release.
Also updates the reference PR for bumping the Go version, since #7981 actually misses a couple of references.
Which issue(s) this PR fixes:
N/A
/area dependency