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 deep-copy pre-commit hook that no longer exists #23244

Merged
merged 1 commit into from
Mar 19, 2016

Conversation

fejta
Copy link
Contributor

@fejta fejta commented Mar 19, 2016

I cannot commit changes because the reference to this deleted code still exists in hooks/pre-commit. It looks like we can just delete it?

#23110 moved deep-copy into run-codegen.sh and #23179 deleted it entirely.

@lavalamp @wojtek-t

@fejta fejta added the priority/important-soon Must be staffed and worked on either currently, or very soon, ideally in time for the next release. label Mar 19, 2016
@k8s-github-robot k8s-github-robot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Mar 19, 2016
@k8s-github-robot
Copy link

Labelling this PR as size/S

@wojtek-t
Copy link
Member

LGTM

@wojtek-t wojtek-t added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Mar 19, 2016
@k8s-bot
Copy link

k8s-bot commented Mar 19, 2016

GCE e2e build/test passed for commit 4fc95c1.

@k8s-github-robot
Copy link

@k8s-bot test this [submit-queue is verifying that this PR is safe to merge]

@k8s-bot
Copy link

k8s-bot commented Mar 19, 2016

GCE e2e build/test passed for commit 4fc95c1.

@k8s-github-robot
Copy link

Automatic merge from submit-queue

k8s-github-robot pushed a commit that referenced this pull request Mar 19, 2016
Auto commit by PR queue bot
@k8s-github-robot k8s-github-robot merged commit 787db61 into kubernetes:master Mar 19, 2016
@fejta fejta deleted the deep branch March 20, 2016 08:19
openshift-publish-robot pushed a commit to openshift/kubernetes that referenced this pull request Jun 26, 2019
remove a revert on the upstream test infrastructure

Origin-commit: d3ae51188b8c78b5f6f17884946d560c5c3cea00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lgtm "Looks good to me", indicates that a PR is ready to be merged. priority/important-soon Must be staffed and worked on either currently, or very soon, ideally in time for the next release. size/S Denotes a PR that changes 10-29 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants