-
Notifications
You must be signed in to change notification settings - Fork 14.6k
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
content/en/case-studies: remove OWNERS #22475
content/en/case-studies: remove OWNERS #22475
Conversation
@nikhita: GitHub didn't allow me to assign the following users: alexcontini. Note that only kubernetes members, repo collaborators and people who have commented on this issue/PR can be assigned. Additionally, issues/PRs can only have 10 assignees at the same time. In response to this:
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. |
/hold
This is concerning because @alexcontini won't be able to lgtm/approve PRs either if they are not a member of the @kubernetes org. |
Deploy preview for kubernetes-io-master-staging ready! Built with commit f3699fc https://deploy-preview-22475--kubernetes-io-master-staging.netlify.app |
442d3e4
to
a779a38
Compare
@nikhita 👋 Coukd you please delete the entire OWNERS file here? Thank you! 🙇 |
As a part of cleaning up inactive members (who haven't been active since the release of v1.11) from OWNERS, this commit removes the OWNERS file for content/en/case-studies.
a779a38
to
f3699fc
Compare
Done ✔️ |
/hold cancel |
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: zacharysarah 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 |
For kubernetes/org#2013
As a part of cleaning up inactive members (who haven't been active since
the release of v1.11) from OWNERS, this commit removes the OWNERS file
for content/en/case-studies.
Original commit message below.
As a part of cleaning up inactive members (who haven't been active since
the release of v1.11) from OWNERS, this commit removes sarahkconway from
the content/en/case-studies/OWNERS file.
Note: With this change, we are only left with one approver for this
directory. We should look at adding more approvers, but this commit
addresses the problem for the short term by ensuring that the bot
doesn't suggest inactive members for approval.
/assign @alexcontini @zacharysarah
cc @mrbobbytables @SarahKConway