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

Propose Bhanu Pulluri as a maintainer #8230

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

matthew1001
Copy link
Contributor

@matthew1001 matthew1001 commented Feb 3, 2025

PR description

Propose Bhanu Pulluri as a Besu maintainer

He has 12 merged PRs and 3 WIP.

Sample of PRs

Bhanu has been actively contributing to a number of areas of Besu. He has a detailed knowledge of the QBFT protocol and has fixed bugs in QBFT and added enhancements that meet the QBFT spec:

He has done various work on JSON/RPC support:

He's added an enhancement to graphql:

He's also resolved high sev CVEs:

Other PRs he's working on include migration of Quorum chains using IBFT "legacy" to QBFT, a complex migrate path that will enable existing enterprise Quorum users to easily move to Besu:

Signed-off-by: Matt Whitehead <[email protected]>
Copy link
Contributor

@pinges pinges left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@pinges pinges left a comment

Choose a reason for hiding this comment

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

LGTM

MAINTAINERS.md Outdated
@@ -9,6 +9,7 @@
| Name | Github | LFID |
| ---------------- | ---------------- | ---------------- |
| Ameziane Hamlat | ahamlat | ahamlat |
| Bhanu Pulluri | pullurib | bpulluri |
Copy link
Contributor

Choose a reason for hiding this comment

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

nit - should be alphabetical by github

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ahh OK, no probs, I'll tweak that now

@matthew1001 matthew1001 changed the title Update MAINTAINERS.md Propose Bhanu Pulluri as a maintainer Feb 19, 2025
@matthew1001
Copy link
Contributor Author

I forgot that the approval process requires an email to the mailing list to inform others about the proposal. I've just sent that email out now.

Copy link
Contributor

@garyschulte garyschulte left a comment

Choose a reason for hiding this comment

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

lgtm

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