Skip to content
This repository was archived by the owner on Jul 11, 2023. It is now read-only.

Add @keithmattix as a codeowner maintainer #4861

Merged

Conversation

trstringer
Copy link
Contributor

@keithmattix has been contributing to the project at the level that is
expected of a maintainer to include: Designs and implementations of
feature work, complex bug triaging and fixing, and indepth and sustained
code reviews raising the quality of the product.

Maintainers, if you agree with this nomination, leave a "+1" comment
and approve the PR (do not merge it).

Signed-off-by: Thomas Stringer [email protected]

Affected area:

Functional Area
Other [X]

Please answer the following questions with yes/no.

  1. Does this change contain code from or inspired by another project? No.

  2. Is this a breaking change? No.

  3. Has documentation corresponding to this change been updated in the osm-docs repo (if applicable)? N/A

@keithmattix has been contributing to the project at the level that is
expected of a maintainer to include: Designs and implementations of
feature work, complex bug triaging and fixing, and indepth and sustained
code reviews raising the quality of the product.

**Maintainers**, if you agree with this nomination, leave a "+1" comment
and approve the PR (do not merge it).

Signed-off-by: Thomas Stringer <[email protected]>
@keithmattix
Copy link
Contributor

Thanks for the nomination Thomas!

Copy link
Contributor

@nojnhuh nojnhuh left a comment

Choose a reason for hiding this comment

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

+1

Copy link
Contributor

@draychev draychev left a comment

Choose a reason for hiding this comment

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

@keithmattix Has been a great driver in the SMI and OSM communities and adding him to the code owners is well deserved.

Copy link
Contributor

@jaellio jaellio left a comment

Choose a reason for hiding this comment

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

+1

@trstringer
Copy link
Contributor Author

This is good to merge as a majority (4/6) of the existing maintainers are in agreement.

@trstringer trstringer merged commit 9d5e442 into openservicemesh:main Jun 30, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants