-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
馃摉 update prerequisites to clarifies go versions which are supported #2548
Conversation
-1 on making this so specific. We can certainly recommend versions that are tested with but I don't think this reflects the reality of what versions are supported. |
Hi @coderanger, We can only support these specific versions. We do not test all ensure it with all versions. These are the versions that we can support now, however, users still able to use any other version as please them. So examples of the issues and scenarios to illustrate it:
I hope that clarifies. |
I'm with @camilamacedo86 here. Declaring the versions which are supported means that this is what the team has identified as potentially broken. You are always welcome to challenge it, but this is not what we recommend. /lgtm |
@AlmogBaku: changing LGTM is restricted to collaborators 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. |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: AlmogBaku, camilamacedo86 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 |
By thinking deeper here. So, that makes no sense. We actually should only have now GO >= 17 and <=18 c/c @AlmogBaku |
Description
馃摉 Update go version information in the QuickStart Prerequisites
Motivation
Closes: #2543