Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Update yaml library. #4499

Merged
merged 1 commit into from
Feb 17, 2015
Merged

Update yaml library. #4499

merged 1 commit into from
Feb 17, 2015
+17 −4

Conversation

vishh
Copy link
Contributor

@vishh vishh commented Feb 17, 2015

Fixes #4041

@vmarmol
Copy link
Contributor

vmarmol commented Feb 17, 2015

LGTM, nice this got in :)

@nikhiljindal
Copy link
Contributor

Will merge when Travis is green.

@nikhiljindal nikhiljindal added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Feb 17, 2015
@nikhiljindal nikhiljindal self-assigned this Feb 17, 2015
nikhiljindal added a commit that referenced this pull request Feb 17, 2015
@nikhiljindal
Merge pull request #4499 from vishh/issue_4041
3dd7c44 
Update yaml library.
@nikhiljindal nikhiljindal merged commit 3dd7c44 into kubernetes:master Feb 17, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@nikhiljindal nikhiljindal

Labels
lgtm "Looks good to me", indicates that a PR is ready to be merged.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

API returns numbers from get that it can't parse when fed into update
3 participants
@vishh @vmarmol @nikhiljindal