-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
Update to Go 1.16 #2003
Update to Go 1.16 #2003
Conversation
We want to wait to at least 1.16.1 to do any upgrade. |
Should I update this PR when 1.16.1 is available? |
@eXeDK sounds good. I don't expect problems, but testing of an imagine built with 1.16.1 would be good as well. |
Sure thing. I'll run the test suite and try building the docker image beforehand. |
This has now been bumped to 1.16.1. |
This has now been bumped to 1.16.2. |
/honk |
In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
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.
Hey, I'd like to avoid the patch version, we had a chat about this in the past and we agreed avoiding it.
I'm happy to merge your PR 🚀
Co-authored-by: Nick Jüttner <nick@juni.io>
Co-authored-by: Nick Jüttner <nick@juni.io>
Co-authored-by: Nick Jüttner <nick@juni.io>
Co-authored-by: Nick Jüttner <nick@juni.io>
Fair enough. I understood you differently I guess. You should be good to go now. Thanks for the suggestions - quick and easy! |
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
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: eXeDK, njuettner 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 |
Description
Updates the go version to 1.16 along with the documentation.
Checklist