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
dependencies: update gh/mitchellh/go-wordwrap #117753
Conversation
/triage accepted |
/cc @dims |
/assign @jpbetz |
/unassign Apologies, but I'm probably not the best person to vet this dependency bump. Happy to review any api-machinery specific aspects. Reassign me if you need me for that specifically. |
cc @liggitt |
go.mod
Outdated
@@ -190,8 +190,8 @@ require ( | |||
github.com/mailru/easyjson v0.7.7 // indirect | |||
github.com/matttproud/golang_protobuf_extensions v1.0.2 // indirect | |||
github.com/mistifyio/go-zfs v2.1.2-0.20190413222219-f784269be439+incompatible // indirect | |||
github.com/mitchellh/go-wordwrap v1.0.0 // indirect | |||
github.com/mitchellh/mapstructure v1.4.1 // indirect |
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.
rather than updating mapstructure, I think we can drop it entirely (we only use it in kube-openapi, opened #118014 and kubernetes/kube-openapi#390 to drop it)
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.
thats indeed better if we can drop it ! thanks @liggitt !
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.
considering we have removed map structure in above referenced PR, I have kept only go-wordwrap
in this PR. ptal.. thanks @liggitt
mitchellh/go-wordwrap@v1.0.0...v1.0.1 Signed-off-by: Humble Chirammal <humble.devassy@gmail.com>
/lgtm |
LGTM label has been added. Git tree hash: 41c38273b224e2cd660d9ba9d486b4e8b608cec4
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: humblec, liggitt 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 |
mitchellh/go-wordwrap@v1.0.0...v1.0.1
/kind cleanup
-->