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

Maintainers: Remove Curators list and move current Curators to emeritus #3885

Merged
merged 6 commits into from
Jun 5, 2022

Conversation

macfarla
Copy link
Contributor

Curator role (formerly non-code maintainer) has been separated out and described here https://wiki.hyperledger.org/display/BESU/Curators

This PR does 2 things

  • move current maintainers that fall under the "non-code" category to emeritus maintainers, since the separate curator role is sufficient for their purposes
  • remove the non-code maintainers section and associated process from the MAINTAINERS.md since this is now defined separately here https://wiki.hyperledger.org/display/BESU/Curators

This PR does not change the process for Besu "code" maintainers, rather removes the extra process so that as a result there is only one type of maintainer.

This PR will remain open for comment for 2 weeks unless a majority of current maintainers votes for the same outcome.

Documentation

  • I thought about documentation and added the doc-change-required label to this PR if
    updates are required.

Changelog

macfarla added 4 commits May 20, 2022 10:40
Signed-off-by: Sally MacFarlane <sally.macfarlane@consensys.net>
Signed-off-by: Sally MacFarlane <sally.macfarlane@consensys.net>
Signed-off-by: Sally MacFarlane <sally.macfarlane@consensys.net>
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@macfarla
Copy link
Contributor Author

@sajz and @MadelineMurray this PR would officially move you from Maintainer to Curator

Copy link
Contributor

@siladu siladu left a comment

Choose a reason for hiding this comment

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

LGTM

@ryjones
Copy link
Member

ryjones commented May 24, 2022

sajz and MadelineMurray this PR would officially move you from Maintainer to Curator

@macfarla From Maintainer to Emeritus

@macfarla
Copy link
Contributor Author

sajz and MadelineMurray this PR would officially move you from Maintainer to Curator

@macfarla From Maintainer to Emeritus

True - the move to Curator is now totally separate to this process

@macfarla macfarla enabled auto-merge (squash) June 5, 2022 22:53
@macfarla macfarla merged commit 03ee5f7 into hyperledger:main Jun 5, 2022
@macfarla macfarla deleted the curator branch July 27, 2022 00:01
eum602 pushed a commit to lacchain/besu that referenced this pull request Nov 3, 2023
…us (hyperledger#3885)

* remove non-code maintainers section

Signed-off-by: Sally MacFarlane <sally.macfarlane@consensys.net>
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.

8 participants