-
Notifications
You must be signed in to change notification settings - Fork 38.7k
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
Cleanup dead code in the codebase #115088
Comments
@thockin: This issue is currently awaiting triage. If a SIG or subproject determines this is a relevant issue, they will accept it by applying the The Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
very willing to oblige |
/assign |
/assign |
I would like to point out that CRI-Dockerd was in fact using some of the "unused" functions in #115103. Just because K8s is not using the function doesn't mean some other repo/project is not. |
@dereknola we don't have guarantees about functions/methods etc. CRI-dockerd is welcome to copy the ones that dropped off from there. |
To be crystal clear - we do not support or endorse importing |
#111654 started here but it got big (shocking!) and the contributor got busy. There are a lot of potential cleanups in there, though, and I hate wasting work.
I see it as an opportunity for someone to rack up some PRs and net-negative LOCs (the best kind!) - just take over that PR, break it up into a bunch of smaller PRs, and send them our way!
A very eager beaver might automate this, but there's a lot of manually-identified low-hanging fruit, I think.
The text was updated successfully, but these errors were encountered: