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.

Sign up for GitHub

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

Jump to bottom

Baseline MAINTAINERS, CODEOWNERS, and external collaborator permissions #101

Closed
Tracked by #125
dblock opened this issue Feb 14, 2023 · 1 comment · Fixed by #113
Closed
Tracked by #125

Baseline MAINTAINERS, CODEOWNERS, and external collaborator permissions #101

dblock opened this issue Feb 14, 2023 · 1 comment · Fixed by #113
Assignees
@macohen

Comments

@dblock
Copy link
Member

dblock commented Feb 14, 2023

Follow opensearch-project/.github#125 to baseline MAINTAINERS, CODEOWNERS, and external collaborator permissions.

Close this issue when:

  1. MAINTAINERS.md has the correct list of project maintainers.
  2. CODEOWNERS exists and has the correct list of aliases.
  3. Repo permissions only contain individual aliases as collaborators with maintain rights, admin, and triage teams.
  4. All other teams are removed from repo permissions.

If this repo's permissions was already baselined, please confirm the above when closing this issue.
@macohen macohen self-assigned this Feb 14, 2023
@dblock dblock mentioned this issue Feb 14, 2023
Closed
87 tasks
@macohen macohen removed the untriaged label Feb 14, 2023
macohen added a commit to macohen/search-processor that referenced this issue Mar 6, 2023
@macohen
modified maintainers as per issue opensearch-project#101
44c5d47 
Signed-off-by: Mark Cohen <[email protected]>
@macohen macohen mentioned this issue Mar 6, 2023
Merged
1 task
msfroh pushed a commit that referenced this issue Mar 6, 2023
@macohen
modified maintainers as per issue #101 (#113)
72e9d48 
Signed-off-by: Mark Cohen <[email protected]>
@dblock
Copy link
Member Author

dblock commented Mar 23, 2023

Verified permissions, looks good!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@macohen macohen

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

modified maintainers as per issue #101 macohen/search-processor
2 participants
@macohen @dblock