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

Move Reiley Yang to emeritus #2198

Merged
merged 2 commits into from
Jun 16, 2023
Merged

Conversation

reyang
Copy link
Member

@reyang reyang commented Jun 15, 2023

It would make sense to remove me from the approvers since I haven't been actively contributing to this project for long time. It is awesome to see the project getting more traction and now we have a good number of active maintainers/approvers well distributed across different companies!! ❤️

CC @jsuereth

@reyang reyang requested a review from a team June 15, 2023 01:03
@codecov
Copy link

codecov bot commented Jun 15, 2023

Codecov Report

Merging #2198 (0c2623e) into main (fc8853e) will decrease coverage by 0.02%.
The diff coverage is n/a.

❗ Current head 0c2623e differs from pull request most recent head eea058f. Consider uploading reports for the commit eea058f to get more accurate results

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #2198      +/-   ##
==========================================
- Coverage   87.52%   87.50%   -0.02%     
==========================================
  Files         169      169              
  Lines        4887     4887              
==========================================
- Hits         4277     4276       -1     
- Misses        610      611       +1     

see 1 file with indirect coverage changes

Copy link
Contributor

@ThomsonTan ThomsonTan 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 @reyang

Copy link
Member

@esigo esigo 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 @reyang

Copy link
Member

@marcalff marcalff 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 @reyang for all your contributions.

@marcalff
Copy link
Member

Keeping the PR opened for 48 hours, to give anyone (not only maintainers/approvers) a chance to comment, ok to merge after that.

@marcalff marcalff merged commit 4cae6aa into open-telemetry:main Jun 16, 2023
@reyang reyang deleted the reyang/emeritus branch June 16, 2023 19:45
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.

6 participants