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 CODEOWNERS, removing inactive contributors and adding newer active contributors #923

Merged
merged 2 commits into from
Jan 9, 2024

Conversation

felixarntz
Copy link
Member

Summary

The CODEOWNERS file has been out of date for quite a while, including several contributors that haven't been active for more than 1 year. This leads to them being unnecessarily pinged, while active contributors may miss out on work their review would be helpful on.

This PR proposes updates to the file, removing any such inactive contributors and suggesting new owners as applicable. Please feel free to suggest additional or alternative code owners as part of your review.

Checklist

  • PR has either [Focus] or Infrastructure label.
  • PR has a [Type] label.
  • PR has a milestone or the no milestone label.

@felixarntz felixarntz added [Type] Documentation Documentation to be added or enhanced Infrastructure Issues for the overall performance plugin infrastructure no milestone PRs that do not have a defined milestone for release [Plugin] Performance Lab Issue relates to work in the Performance Lab Plugin only labels Jan 8, 2024
Comment on lines +43 to +45
/modules/database/audit-autoloaded-options @manuelRod @felixarntz
/tests/modules/database/audit-autoloaded-options @manuelRod @felixarntz
/tests/testdata/modules/database/audit-autoloaded-options @manuelRod @felixarntz
Copy link
Member Author

Choose a reason for hiding this comment

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

There is #889 left to do as a crucial enhancement to this module, so potentially the person who picks up that issue would be a great candidate to include as additional code owner here.

@joemcgill
Copy link
Member

Thanks @felixarntz. I think this is a great idea. I'm happy to be added as a codeowner for the images and measurement focuses as well, if you'd like.

@felixarntz
Copy link
Member Author

@joemcgill Awesome! done in 5d2f498

@felixarntz felixarntz requested a review from joemcgill January 8, 2024 17:41
@mukeshpanchal27 mukeshpanchal27 merged commit d259c43 into trunk Jan 9, 2024
4 checks passed
@mukeshpanchal27 mukeshpanchal27 deleted the update/codeowners branch January 9, 2024 04:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Infrastructure Issues for the overall performance plugin infrastructure no milestone PRs that do not have a defined milestone for release [Plugin] Performance Lab Issue relates to work in the Performance Lab Plugin only [Type] Documentation Documentation to be added or enhanced
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants