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

org: Transition inactive maintainers to Emeritus (2024) #1530

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

craigbox
Copy link
Contributor

This PR moves maintainers who appear to be inactive in the project to Emeritus.

If a maintainer becomes inactive in the project for over a year, the individual will be asked to transition to an emeritus maintainer. Emeritus maintainers lose their ability to approve code contributions, but retain their voting rights for up to one year. After one year, emeritus maintainers revert back to being normal members with no voting rights.

This has a few motivations:

  • Thank and honor all the past maintainers for all the invaluable work they have done over the years
  • Trim the list of maintainers to (1) improve discover-ability of who is able to actively maintainer/review/etc an area and (2) reduce the number of privileged users in the org

This list was compiled by examining maintainers activity (PRs, issues, comments, etc) over the past year. This list is those who appear to no longer be involved; another PR will follow with members who are present but have low activity.

The goal here is very much not to "kick people out" of the maintainers list. If you feel you are interested in continuing your maintainer role, simply comment here (or DM me) and I will update the PR. Given it's close to the holiday period I will keep this open until mid-January or until I've heard from everyone on the list. However, one note: you do not need to hold a formal title of "maintainer" to be a valuable contributor to the project, and if you do step down the process to step back into the role when you have more time to commit to Istio is fairly quick and easy.

Tagging impacted members: @stewartbutler, @davidhauck, @richardwxn, @adiprerepa, @lizan, @shankgan

Thanks again for all your contributions!

Past PRs: #817 (2022), #1322 (2023)

@craigbox craigbox requested a review from a team as a code owner December 19, 2024 21:14
@istio-testing istio-testing added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Dec 19, 2024
@kfaseela
Copy link
Member

LGTM

@craigbox craigbox added the do-not-merge Block automatic merging of a PR. label Dec 19, 2024
Copy link
Member

@howardjohn howardjohn left a comment

Choose a reason for hiding this comment

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

Thank you all!

@hzxuzhonghu
Copy link
Member

LGTM

1 similar comment
@kebe7jun
Copy link
Member

LGTM

@lizan
Copy link
Contributor

lizan commented Dec 24, 2024

LGTM, Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
do-not-merge Block automatic merging of a PR. size/S Denotes a PR that changes 10-29 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants