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

Add @shalier as a codeowner maintainer #5261

Merged

Conversation

trstringer
Copy link
Contributor

@shalier has been contributing to the project at the level that is expected of a maintainer.

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

Signed-off-by: Thomas Stringer thomas@trstringer.com

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

@shalier has been contributing to the project at the level that is
expected of a maintainer.

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

Signed-off-by: Thomas Stringer <thomas@trstringer.com>
Copy link
Contributor

@keithmattix keithmattix left a comment

Choose a reason for hiding this comment

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

+1

@jsturtevant
Copy link
Contributor

+1

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

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

@trstringer trstringer merged commit 7f57762 into openservicemesh:main Nov 30, 2022
keithmattix pushed a commit to keithmattix/osm that referenced this pull request Dec 8, 2022
@shalier has been contributing to the project at the level that is
expected of a maintainer.

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

Signed-off-by: Thomas Stringer <thomas@trstringer.com>
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