Remove unused files #7643
Remove unused files #7643
Conversation
Signed-off-by: Mathew Wicks <5735406+thesuperzapper@users.noreply.github.com>
|
@kimwnasptd if you agree with all these removals, please LGTM. After that, @andreyvelich can approve (it requires a root approver). |
There was a problem hiding this comment.
Thank you for this @thesuperzapper!
Overall, lgtm.
Should we also remove security folder: https://github.com/thesuperzapper/kubeflow/tree/remove-unused-files/security @juliusvonkohout @akgraner ?
/assign @kubeflow/wg-manifests-leads @kubeflow/wg-notebooks-leads @kubeflow/kubeflow-steering-committee
|
We should probably move the security folder to kubeflow/manifests. But this can be done in another PR. |
|
@juliusvonkohout Can you submit PR to move it under manifests repo and remove it from |
|
@andreyvelich @juliusvonkohout I think security info should be under |
I am fine with the both as long as maintainers can take care of those assets. |
Signed-off-by: Mathew Wicks <5735406+thesuperzapper@users.noreply.github.com>
|
@andreyvelich I have also removed the |
There was a problem hiding this comment.
Thanks @thesuperzapper!
/lgtm
/assign @kubeflow/kubeflow-steering-committee
|
/lgtm |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: terrytangyuan The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
As part of migrating code out of this repo #7549, there is a bunch of code on this repo which does nothing anymore.
This PR removes the following content from the
kubeflow/kubeflowrepo:./docs/gke/:./docs_dev/./py/:./security/:kubeflow/community, see:kubeflow/kubeflowcommunity#749./testing/gh-actions/.pylintrc.style.yapfprow_config.yaml