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

remove duplicate import with different name #602

Merged
merged 1 commit into from
May 15, 2017

Conversation

surajssd
Copy link
Member

k8s.io/kubernetes/pkg/api is imported directly and once with the alias import name kapi so remove the original import and keep the aliased with kapi one and also replace the code bits which call it with api name.

`k8s.io/kubernetes/pkg/api` is imported directly and once with the alias
import name `kapi` so remove the original import and keep the aliased with
`kapi` one and also replace the code bits which call it with `api` name.
@k8s-ci-robot k8s-ci-robot added the cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. label May 15, 2017
@cdrage
Copy link
Member

cdrage commented May 15, 2017

LGTM.

@surajssd
Copy link
Member Author

@cdrage you will have to approve it to merge it!

@cdrage
Copy link
Member

cdrage commented May 15, 2017

@surajssd Got to wait for one more approval.

@surajnarwade @kadel @containscafeine @rtnpro ?

@surajssd
Copy link
Member Author

@cdrage thanks 👍

@surajssd surajssd merged commit b877014 into kubernetes:master May 15, 2017
@cdrage
Copy link
Member

cdrage commented May 15, 2017

@surajssd Try to wait until 2 people approve it 👍 before any merges :)

@surajssd
Copy link
Member Author

@cdrage sure ! I thought this is smaller one, with mainly name changes so merged it!

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.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants