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.

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

Migrate to client-go #463

Closed
wants to merge 8 commits into from
Closed

Migrate to client-go #463

wants to merge 8 commits into from

Conversation

aledbf
Copy link
Member

@aledbf aledbf commented Mar 17, 2017

No description provided.

@k8s-ci-robot k8s-ci-robot added the cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. label Mar 17, 2017
@k8s-reviewable
Copy link

This change is Reviewable

@aledbf aledbf force-pushed the update-deps branch 3 times, most recently from 25814e9 to 1a5c2d3 Compare March 27, 2017 01:06
@aledbf aledbf mentioned this pull request Mar 29, 2017
5 tasks
@aledbf
Copy link
Member Author

aledbf commented Mar 30, 2017

@nicksardo please check the second commit and tell me if this refactoring is ok?

@aledbf aledbf force-pushed the update-deps branch 2 times, most recently from 4caaa12 to 42d2e4d Compare March 30, 2017 21:43
@coveralls
Copy link

Coverage Status

Coverage increased (+0.3%) to 44.123% when pulling f5050b91abfee58a058b4a4df4c59f0f2aec7a93 on aledbf:update-deps into 0bdf253 on kubernetes:master.

@aledbf aledbf changed the title WIP: Migrate to client-go [do not merge] Migrate to client-go Mar 30, 2017
@aledbf aledbf requested a review from nicksardo March 30, 2017 22:24
@coveralls
Copy link

Coverage Status

Coverage increased (+0.3%) to 44.14% when pulling aa26719 on aledbf:update-deps into 0bdf253 on kubernetes:master.

@nicksardo
Copy link
Contributor

Currently on vacation; I'll look at this on Sunday.

@aledbf aledbf mentioned this pull request Apr 1, 2017
@aledbf aledbf deleted the update-deps branch June 13, 2017 12:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. nginx
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants