Skip to content

chore: promote @NameHaibinZhang to the maintainer#498

Open
y1yang0 wants to merge 2 commits into
open-telemetry:mainfrom
y1yang0:yyang/promote
Open

chore: promote @NameHaibinZhang to the maintainer#498
y1yang0 wants to merge 2 commits into
open-telemetry:mainfrom
y1yang0:yyang/promote

Conversation

@y1yang0
Copy link
Copy Markdown
Contributor

@y1yang0 y1yang0 commented May 13, 2026

I'd like to formally nominate @Haibin Zhang as a maintainer for opentelemetry-go-compile-instrumentation.

He is a core developer of our go-compile auto-instrumentation and has been making steady contributions: CommitList.

He’s a regular attendee at our weekly meetings (though he tends to listen more due to the language barrier). As part of the team led by @Huxing Zhang, he and I have been working closely to build cloud-based Golang observability.

Looking ahead, he is committed to contributing to the auto-instrumentation plugins long-term and will bring a lot of valuable insights to improve the tool module.

Would love to hear your thoughts on adding him to the maintainers list!

Copilot AI review requested due to automatic review settings May 13, 2026 07:26
@y1yang0 y1yang0 requested a review from a team as a code owner May 13, 2026 07:26
@github-actions github-actions Bot added the scope:chore Something that needs to be taken care of but not very appealing label May 13, 2026
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Promotes Haibin Zhang from Approver to Maintainer in the project's governance list, and consolidates the multiple per-role "for more information" links into a single reference.

Changes:

  • Add Haibin Zhang to the Maintainers list.
  • Remove the now-empty Approvers entry for Haibin Zhang.
  • Consolidate three separate role-info links (and a duplicated emeritus link) into one combined link.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Copy Markdown
Member

@kakkoyun kakkoyun left a comment

Choose a reason for hiding this comment

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

LGTM 🎉

nit: Let's not remove the explanation lines.

Comment thread README.md
Comment thread README.md
@y1yang0
Copy link
Copy Markdown
Contributor Author

y1yang0 commented May 18, 2026

Thank you @ralf0131 @darccio @txabman42 @kakkoyun all for the reviews. Really appreciate.

Comment thread README.md
For more information about the emeritus role, see the
[community repository](https://github.com/open-telemetry/community/blob/main/guides/contributor/membership.md#emeritus-maintainerapprovertriager).

For more information about the emeritus role, see the [community repository](https://github.com/open-telemetry/community/blob/main/guides/contributor/membership.md#emeritus-maintainerapprovertriager).
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I remove this line as it's duplicated with previous one.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

scope:chore Something that needs to be taken care of but not very appealing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants