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

chore: update component owner for AWS to jj22ee #2506

Merged
merged 3 commits into from
Nov 5, 2024

Conversation

jj22ee
Copy link
Contributor

@jj22ee jj22ee commented Oct 25, 2024

Short description of the changes

Discussed within the X-Ray team at AWS, I will take over ownership+maintenance duties of the AWS components from @carolabadeer, whose bandwidth is currently being focused outside of OTel.

Copy link

codecov bot commented Oct 25, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 90.86%. Comparing base (1165264) to head (af64430).
Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2506   +/-   ##
=======================================
  Coverage   90.86%   90.86%           
=======================================
  Files         161      161           
  Lines        7858     7858           
  Branches     1612     1612           
=======================================
  Hits         7140     7140           
  Misses        718      718           

Copy link
Contributor

This package does not have an assigned component owner and is considered unmaintained. As such this package is in feature-freeze and this PR will be closed with 14 days unless a new owner or a sponsor (a member of @open-telemetry/javascript-approvers) for the feature is found. It is the responsibility of the author to find a sponsor for this feature.
Are you familiar with this package? Consider becoming a component owner.

Copy link
Member

@pichlermarc pichlermarc left a comment

Choose a reason for hiding this comment

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

Awesome, welcome @jj22ee - happy to have you on board 🎉

@pichlermarc pichlermarc merged commit 95547b8 into open-telemetry:main Nov 5, 2024
23 checks passed
@pichlermarc
Copy link
Member

@jj22ee have you considered joining the OTel Organization? That would allow us to auto-assign pull requests to you for the relevant packages. I'd gladly sponsor you joining the Org. :)

@jj22ee
Copy link
Contributor Author

jj22ee commented Nov 26, 2024

@pichlermarc Hey, thanks for approving this code-owner update. I do want to join the OTel Organization, thanks for offering to sponsor me to join. Please let me know what I need to do on my end to join.

@pichlermarc
Copy link
Member

@jj22ee great to hear that. 🙌 Usually two sponsors from the organization are needed (from two different companies) - so you'd need to find one more person to sponsor you. You can find all the requirements to become a member in the community repo.

Once you have found a second sponsor and opened the request as outlined in the doc I linked, you can ping me on the CNCF slack and I'll comment to say that I support your membership request. 🙂

@trentm
Copy link
Contributor

trentm commented Nov 27, 2024

Hi @jj22ee, I'm happy to be a second sponsor for you.

@jj22ee
Copy link
Contributor Author

jj22ee commented Nov 27, 2024

Thanks @trentm!

I will follow-up soon to create the membership request.

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

Successfully merging this pull request may close these issues.

5 participants