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

Updated vendor with service catalog and client-go #716

Merged
merged 4 commits into from
Sep 11, 2018

Conversation

surajnarwade
Copy link
Contributor

Updated vendor with:

service-catalog: v0.1.31
openshift/client-go: 431ec9a26e5021f35fa41ee9a89842db9bfdb370

@surajnarwade
Copy link
Contributor Author

I was playing with vendor and this seems to work I guess
cc @mik-dass @dgolovin

Copy link
Contributor

@anmolbabu anmolbabu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I see the build goes through hence approving

Copy link
Member

@syamgk syamgk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

build went successful on my machine, LGTM

@surajnarwade
Copy link
Contributor Author

@dgolovin , @mik-dass , Please test git clone on windows after merging this

@surajnarwade
Copy link
Contributor Author

Only merge after testing it on windows

Copy link
Contributor

@mik-dass mik-dass left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@surajnarwade Tested on windows and git clone and go build was successfully. Rest also LGTM.

@mik-dass
Copy link
Contributor

@cdrage please review and merge

@cdrage
Copy link
Member

cdrage commented Sep 11, 2018

This LGTM.

@cdrage cdrage merged commit 64f9db2 into redhat-developer:master Sep 11, 2018
mik-dass pushed a commit to mik-dass/odo that referenced this pull request Oct 17, 2018
* Updated client-go, service-catalog

* updated vendor

* Removed go-github and k8s.io/api to resolve conflicts

* updated vendor
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants