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

Update Maintainer.md to match CODEOWNER #3831

Merged
merged 1 commit into from
Apr 20, 2023

Conversation

ananzh
Copy link
Member

@ananzh ananzh commented Apr 20, 2023

Description

Based on requirement here:
opensearch-project/.github#125

CODEOWNER should include a subset or all maintainers. This PR updates the original CODEOWNER PR here #3639 based on the request.

Issue Resolve
#2878

Checklist

  • By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license and subject to the Developers Certificate of Origin.
    For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@ananzh ananzh changed the title Update CODEOWNER to match maintainer Update Maintainer.md to match CODEOWNER Apr 20, 2023
Based on requirement here:
opensearch-project/.github#125

CODEOWNER should include a subset or all maintainers.
Based on the request, update maintainers.

Issue Resolve
opensearch-project#2878

Signed-off-by: Anan Zhuang <ananzh@amazon.com>
Copy link
Collaborator

@kolchfa-aws kolchfa-aws left a comment

Choose a reason for hiding this comment

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

LGTM

@Naarcha-AWS Naarcha-AWS merged commit f9a966a into opensearch-project:main Apr 20, 2023
@ananzh
Copy link
Member Author

ananzh commented Apr 20, 2023

@opensearch-project/admin and @gaiksaya I have confirmed with @Naarcha-AWS updated maintainers. Now MAINTAINERS and CODEOWNERS are both correct. Pls help to double check the permissions. Thanks.

@ananzh ananzh mentioned this pull request Apr 20, 2023
1 task
vagimeli pushed a commit that referenced this pull request Apr 25, 2023
Based on requirement here:
opensearch-project/.github#125

CODEOWNER should include a subset or all maintainers.
Based on the request, update maintainers.

Issue Resolve
#2878

Signed-off-by: Anan Zhuang <ananzh@amazon.com>
vagimeli pushed a commit that referenced this pull request May 4, 2023
Based on requirement here:
opensearch-project/.github#125

CODEOWNER should include a subset or all maintainers.
Based on the request, update maintainers.

Issue Resolve
#2878

Signed-off-by: Anan Zhuang <ananzh@amazon.com>
vagimeli added a commit that referenced this pull request May 4, 2023
harshavamsi pushed a commit to harshavamsi/documentation-website that referenced this pull request Oct 31, 2023
Based on requirement here:
opensearch-project/.github#125

CODEOWNER should include a subset or all maintainers.
Based on the request, update maintainers.

Issue Resolve
opensearch-project#2878

Signed-off-by: Anan Zhuang <ananzh@amazon.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants