-
Notifications
You must be signed in to change notification settings - Fork 61
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 go to 1.21 #204
Upgrade go to 1.21 #204
Conversation
Signed-off-by: Tim Ramlot <42113979+inteon@users.noreply.github.com>
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.
/lgtm
But we might consider requiring just 1.21.0 in the go.mod files. Do we plan to bump the requirement as new Go releases are released?
I don't think we have to bump it every time, I just did it now because in 1.21 the go.sum files contain more entries (I think they were missing in 1.20). So 1.21.0 would work too. |
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: erikgb, inteon 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 |
Closing the loop: https://go.dev/ref/mod#go-mod-file-go The go directive is specifying a minimum. The fact that it's an older version here isn't a problem really! |
Upgrade all go versions & go.mod files to 1.21