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

Switch to using github.com/openshift/imagebuilder #10216

Merged
merged 5 commits into from Aug 4, 2016

Conversation

smarterclayton
Copy link
Contributor

@smarterclayton smarterclayton commented Aug 4, 2016

Also cleanup some Godeps incorrectness that crept in after a Godep save

@smarterclayton
Copy link
Contributor Author

[test] @liggitt please take a look at the bump changes. I've already pushed stable-20160804 (shame this isn't the year 3216)

@liggitt
Copy link
Contributor

liggitt commented Aug 4, 2016

bump LGTM, I was surprised to see github.com/onsi/ginkgo/ginkgo and github.com/jteeuwen/go-bindata/go-bindata leave Godeps.json

@smarterclayton
Copy link
Contributor Author

Neither of those are actually included in the dependency tree - they are binaries we invoke.

@smarterclayton
Copy link
Contributor Author

So godep save can't save them.

@smarterclayton
Copy link
Contributor Author

[test] #9938

@smarterclayton
Copy link
Contributor Author

[merge]

@openshift-bot
Copy link
Contributor

Evaluated for origin test up to d778fd1

@openshift-bot
Copy link
Contributor

openshift-bot commented Aug 4, 2016

continuous-integration/openshift-jenkins/merge SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pr_origin/7518/) (Image: devenv-rhel7_4757)

@openshift-bot
Copy link
Contributor

Evaluated for origin merge up to d778fd1

@openshift-bot
Copy link
Contributor

continuous-integration/openshift-jenkins/test SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pr_origin/7518/)

@openshift-bot openshift-bot merged commit c63d94f into openshift:master Aug 4, 2016
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

3 participants