-
Notifications
You must be signed in to change notification settings - Fork 38.7k
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
Switch to sigs.k8s.io/yaml from ghodss/yaml #70598
Switch to sigs.k8s.io/yaml from ghodss/yaml #70598
Conversation
/test pull-kubernetes-integration |
/priority important-soon |
/test pull-kubernetes-integration |
/lgtm |
/assign @thockin |
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
/approve
/assign @thockin |
/assign @jbeda |
/approve |
5ad8b05
to
e5bbee0
Compare
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.
rebase required
/approve
Godeps/Godeps.json
Outdated
@@ -26,12 +26,12 @@ | |||
}, | |||
{ | |||
"ImportPath": "cloud.google.com/go/compute/metadata", | |||
"Comment": "v0.1.0-115-g3b1ae45", | |||
"Comment": "v0.1.0-115-g3b1ae453", |
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.
Can we remove these comment changes? (will hopefully be resolved once and for all soon)
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.
Done!
Change-Id: Ic72b5131bf441d159012d67a6a3d87088d0e6d31
e5bbee0
to
43f523d
Compare
/lgtm 🚀 🏆 |
@smarterclayton @liggitt @thockin - can you please |
/approve |
/retest Review the full test history for this PR. Silence the bot with an |
2 similar comments
/retest Review the full test history for this PR. Silence the bot with an |
/retest Review the full test history for this PR. Silence the bot with an |
Change-Id: Ic72b5131bf441d159012d67a6a3d87088d0e6d31
What type of PR is this?
/kind cleanup
What this PR does / why we need it:
We now have a new repository named
sigs.k8s.io/yaml
which has the code forked fromghodss/yaml
Which issue(s) this PR fixes (optional, in
fixes #<issue number>(, fixes #<issue_number>, ...)
format, will close the issue(s) when PR gets merged):Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?: