-
Notifications
You must be signed in to change notification settings - Fork 508
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
🌱 CODEOWNERS: Add Stephen Augustus (justaugustus) as maintainer #1530
Conversation
Integration tests success for |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks and welcome :)
Thanks so much, Azeem!! 🎉 |
Codecov Report
@@ Coverage Diff @@
## main #1530 +/- ##
=======================================
Coverage 52.47% 52.47%
=======================================
Files 68 68
Lines 6131 6131
=======================================
Hits 3217 3217
Misses 2707 2707
Partials 207 207 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @justaugustus!
.github/workflows/* CODEOWNERS are effectively maintainers, but with the current configuration, they are not being automatically tagged for review for other file changes. Here we simplify to `*`, in preparation for adding additional maintainers. (Maintainers have also been alpha-sorted.) Signed-off-by: Stephen Augustus <[email protected]>
Signed-off-by: Stephen Augustus <[email protected]>
7efa004
to
1f04488
Compare
Integration tests success for |
Please check if the PR fulfills these requirements
What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
Part of community: Is there a contributor ladder? #1529.
CODEOWNERS: Simplify maintainers
.github/workflows/* CODEOWNERS are effectively maintainers, but
with the current configuration, they are not being automatically
tagged for review for other file changes.
Here we simplify to
*
, in preparation for adding additionalmaintainers.
(Maintainers have also been alpha-sorted.)
CODEOWNERS: Add Stephen Augustus (justaugustus) as maintainer
Signed-off-by: Stephen Augustus [email protected]
cc: @inferno-chromium @naveensrinivasan @azeemshaikh38 @laurentsimon
What is the current behavior? (You can also link to an open issue here)
N/A
What is the new behavior (if this is a feature change)?
MOAR MAINTAINERS!
Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)
Hopefully not 🙃
Other information: