-
Notifications
You must be signed in to change notification settings - Fork 40
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
build: update go 1.16 to 1.17 #129
Conversation
Signed-off-by: Nitin Goyal <nigoyal@redhat.com>
@iamniting is it tested for PVC operations or need to be tested? |
@@ -20,3 +20,78 @@ require ( | |||
sigs.k8s.io/controller-runtime v0.10.2 | |||
sigs.k8s.io/yaml v1.2.0 | |||
) | |||
|
|||
require ( |
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.
- any idea why this was factored out into a new
require
block and no changes togo.sum
?
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.
Actually, I ran go mod tidy
which did this change, and nothing in the go.mod file. As they are all indirect deps we might already have all of them in the go.sum
Not yet |
Tested it out and it works well
|
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: iamniting, leelavg, nbalacha 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 |
Signed-off-by: Nitin Goyal nigoyal@redhat.com