-
Notifications
You must be signed in to change notification settings - Fork 954
Add unmaintained notice to contrib components #2313
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
Add unmaintained notice to contrib components #2313
Conversation
Signed-off-by: Anna Jung (VMware) <[email protected]>
The OWNERS file is not available, therefore, am not sure who originally created the component. cc @kimwnasptd |
To remove the unmaintained label, check out the `missing requirements #2311 and leave a comment on the issue to indicate that you'll take action to provide the missing requirements and are willing to keep the component up to date. Thank you! |
The message looks good. @annajung I'm guessing we are going to use the same for the other READMEs as well right? I.e. https://github.com/kubeflow/manifests/tree/master/contrib/feast |
@kimwnasptd Yes, that's the plan. I'm creating a separate PR for each component to ensure that the relevant components' owners are tagged. |
Signed-off-by: Anna Jung (VMware) <[email protected]>
Signed-off-by: Anna Jung (VMware) <[email protected]>
Signed-off-by: Anna Jung (VMware) <[email protected]>
Signed-off-by: Anna Jung (VMware) <[email protected]>
cb7b6d0
to
3c54a7e
Compare
Signed-off-by: Anna Jung (VMware) <[email protected]>
3c54a7e
to
62dd45c
Compare
@kimwnasptd Update: The components that did not have the OWNERS file are merged into this PR now. Rest that requires tagging owners for awareness will be a separate PR |
/retitle Add unmaintained notice to contrib components |
I like the approach of first tackling the components without OWNERs. Then we can move on with the unmaintained ones. /lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: annajung, kimwnasptd 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 |
* Add unmaintained notice to contrib basic-auth Signed-off-by: Anna Jung (VMware) <[email protected]> * Add unmaintained notice to contrib experimental Signed-off-by: Anna Jung (VMware) <[email protected]> * Add unmaintained notice to contrib application Signed-off-by: Anna Jung (VMware) <[email protected]> * Add unmaintained notice to contrib spartakus Signed-off-by: Anna Jung (VMware) <[email protected]> * Add unmaintained notice to contrib modeldb Signed-off-by: Anna Jung (VMware) <[email protected]> * Add unmaintained notice to contrib gatekeeper Signed-off-by: Anna Jung (VMware) <[email protected]> Signed-off-by: Anna Jung (VMware) <[email protected]>
Signed-off-by: Anna Jung (VMware) [email protected]
Which issue is resolved by this Pull Request:
Part of #2311
Description of your changes:
/contrib/application
/contrib/basic-auth
/contrib/experimental
/contrib/gatekeeper
/contrib/modeldb
/contrib/spartakus
cc @kimwnasptd @kubeflow/wg-manifests-leads