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

fix: Move maintainers to emeritus #5743

Merged
merged 6 commits into from
Aug 17, 2023
Merged

Conversation

macfarla
Copy link
Contributor

@macfarla macfarla commented Aug 2, 2023

I propose moving maintainers @mark-terry and @wcgcyx to Emeritus status, pursuant to the inactivity clause. These maintainers have had no activity since January 2023.

We very much appreciate their contributions, but moving their status to emeritus (and thus revoking PR approval privileges) is in the interest of an orderly project. If any of these maintainers express in this PR that they intend to make contributions in the next quarter, then they will not be moved to emeritus status.

I propose this vote remain open until either both maintainers mentioned above confirm on this PR their acceptance of this change, OR an absolute majority of active maintainers votes for the same outcome, OR until two weeks has passed, after which a voting majority will determine the outcome (with a tie resulting in no change).

Signed-off-by: Sally MacFarlane <macfarla.github@gmail.com>
@github-actions
Copy link

github-actions bot commented Aug 2, 2023

  • I thought about documentation and added the doc-change-required label to this PR if updates are required.
  • I thought about the changelog and included a changelog update if required.
  • If my PR includes database changes (e.g. KeyValueSegmentIdentifier) I have thought about compatibility and performed forwards and backwards compatibility tests

@macfarla macfarla changed the title fix: Remove maintainer fix: Move maintainers to emeritus Aug 2, 2023
@daniellehrner
Copy link
Contributor

@macfarla In the PR the maintainers are removed from the table of the active maintainers, but they are not added to the emeritus maintainer table. If I am not mistaken in the past those were moved there, weren't they?

@macfarla
Copy link
Contributor Author

macfarla commented Aug 3, 2023

@macfarla In the PR the maintainers are removed from the table of the active maintainers, but they are not added to the emeritus maintainer table. If I am not mistaken in the past those were moved there, weren't they?

you are absolutely correct. thanks for picking this up - fixed.

Copy link
Contributor

@wcgcyx wcgcyx left a comment

Choose a reason for hiding this comment

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

LGTM!

@macfarla macfarla enabled auto-merge (squash) August 16, 2023 23:30
@macfarla macfarla merged commit 18975c7 into hyperledger:main Aug 17, 2023
garyschulte pushed a commit to garyschulte/besu that referenced this pull request Aug 28, 2023
* moved to emeritus

Signed-off-by: Sally MacFarlane <macfarla.github@gmail.com>

---------

Signed-off-by: Sally MacFarlane <macfarla.github@gmail.com>
garyschulte pushed a commit to garyschulte/besu that referenced this pull request Aug 28, 2023
* moved to emeritus

Signed-off-by: Sally MacFarlane <macfarla.github@gmail.com>

---------

Signed-off-by: Sally MacFarlane <macfarla.github@gmail.com>
garyschulte pushed a commit to garyschulte/besu that referenced this pull request Aug 28, 2023
* moved to emeritus

Signed-off-by: Sally MacFarlane <macfarla.github@gmail.com>

---------

Signed-off-by: Sally MacFarlane <macfarla.github@gmail.com>
Signed-off-by: garyschulte <garyschulte@gmail.com>
@macfarla macfarla deleted the rm-maintainer branch September 13, 2023 06:00
eum602 pushed a commit to lacchain/besu that referenced this pull request Nov 3, 2023
* moved to emeritus

Signed-off-by: Sally MacFarlane <macfarla.github@gmail.com>

---------

Signed-off-by: Sally MacFarlane <macfarla.github@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

9 participants