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.

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

baseline repo groups #174

Merged
merged 2 commits into from
Apr 26, 2023
Merged
Show file tree
Hide file tree
Changes from 1 commit
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 1 addition & 3 deletions .github/CODEOWNERS
Validating CODEOWNERS rules …
Original file line number Diff line number Diff line change
@@ -1,4 +1,2 @@


# This should match the owning team set up in https://github.com/orgs/opensearch-project/teams
**/* @opensearch-project/dashboards-visualizations
* @anirudha @nknize @ps48 @kavithacm @joshuali925 @YANG-DB @davidcui1225 @rupal-bq @CEHENKLE @mengweieric @eugenesk24 @vamsi-amazon
14 changes: 10 additions & 4 deletions MAINTAINERS.md
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,13 @@ This document contains a list of maintainers in this repo. See [opensearch-proje

## Current Maintainers

| Maintainer | GitHub ID | Affiliation |
| ---------- | ----------------------------------------------- | ----------- |
| David Cui | [davidcui1225](https://github.com/davidcui1225) | Amazon |
| Joshua Li | [joshuali925](https://github.com/joshuali925) | Amazon |
| Maintainer | GitHub ID | Affiliation |
| ----------------- | ------------------------------------------------- | ----------- |
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we might need a vote to get @ps48 or @joshuali925 onto this list

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's make sure the PR code errors in codeowners file are expected.



    Unknown owner on line 2: make sure @pjfitzgibbons exists and has write access to the repository

    *   @pjfitzgibbons @ps48 @kavithacm @derek-ho @joshuali925 @dai-chen @YANG-DB @rupal-bq @mengweieric @vamsi-amazon @swiddis @penghuo @seankao-az

    Unknown owner on line 2: make sure @derek-ho exists and has write access to the repository

    …jfitzgibbons @ps48 @kavithacm @derek-ho @joshuali925 @dai-chen @YANG-DB @rupal-bq @mengweieric @vamsi-amazon @swiddis @penghuo @seankao-az

    Unknown owner on line 2: make sure @dai-chen exists and has write access to the repository

    …ithacm @derek-ho @joshuali925 @dai-chen @YANG-DB @rupal-bq @mengweieric @vamsi-amazon @swiddis @penghuo @seankao-az

    Unknown owner on line 2: make sure @swiddis exists and has write access to the repository

    …bq @mengweieric @vamsi-amazon @swiddis @penghuo @seankao-az

    Unknown owner on line 2: make sure @penghuo exists and has write access to the repository

    …eieric @vamsi-amazon @swiddis @penghuo @seankao-az

    Unknown owner on line 2: make sure @seankao-az exists and has write access to the repository

    …amsi-amazon @swiddis @penghuo @seankao-az


## Emeritus Maintainers

| Maintainer | GitHub ID | Affiliation |
| ----------------- | ------------------------------------------------------- | ----------- |
| Charlotte Henkle | [CEHENKLE](https://github.com/CEHENKLE) | Amazon |
| Anirudha Jadhav | [anirudha](https://github.com/anirudha) | Amazon |
| Nick Knize | [nknize](https://github.com/nknize) | Amazon |