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

contributors/devel: update godeps instructions with required hack scripts #591

Merged
merged 1 commit into from
May 1, 2017

Conversation

ericchiang
Copy link
Contributor

Work I had to do to get kubernetes/kubernetes#45056 working.

cc @sttts @thockin @caesarxuchao

@k8s-ci-robot k8s-ci-robot added the cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. label Apr 28, 2017
@eparis eparis merged commit 7f72640 into master May 1, 2017
@ericchiang ericchiang deleted the update-godep-instructions branch May 1, 2017 17:27
@sttts
Copy link
Contributor

sttts commented May 2, 2017

lgtm

# Regenerate removed BUILD, licenses, and client-go Godeps files.
./hack/update-bazel.sh
./hack/update-godeps-licenses.sh
./hack/update-staging-client-go.sh
Copy link
Member

Choose a reason for hiding this comment

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

@caesarxuchao I am worried that we are documenting this for contributors...

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.

5 participants