Skip to content

Conversation

@yiyuan-he
Copy link
Contributor

Short description of the changes

  • Adding myself as a component owner of the X-Ray Remote Sampler per request of existing owner @jj22ee

@yiyuan-he yiyuan-he requested a review from a team as a code owner March 18, 2025 17:33
@linux-foundation-easycla
Copy link

linux-foundation-easycla bot commented Mar 18, 2025

CLA Signed

The committers listed above are authorized under a signed CLA.

@github-actions github-actions bot requested a review from jj22ee March 18, 2025 17:38
@yiyuan-he yiyuan-he changed the title docs: add new x-ray remote sampler owner [WIP] docs: add new x-ray remote sampler owner Mar 18, 2025
@yiyuan-he yiyuan-he changed the title docs: add new x-ray remote sampler owner chore(sampler-aws-xray): add @yiyuanh-he as component owner Mar 19, 2025
@pichlermarc
Copy link
Member

Hi @yiyuan-he - we'll need to get you onboarded as a member of the OTel org in order for you to take ownership of the component. I'll sponsor you but you'll will need at least one other sponsor that meets the requirements stated in the doc.

@yiyuan-he
Copy link
Contributor Author

Thanks @pichlermarc - I really appreciate your offer to sponsor my membership to the OTel org.

I currently have an open PR in the Ruby contrib repo. I'll post a comment there to see if any of the reviewers would be willing to be my second sponsor.

@codecov
Copy link

codecov bot commented Apr 4, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 89.50%. Comparing base (8838636) to head (0f6f48c).
Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2761      +/-   ##
==========================================
+ Coverage   89.45%   89.50%   +0.04%     
==========================================
  Files         174      174              
  Lines        8318     8318              
  Branches     1591     1591              
==========================================
+ Hits         7441     7445       +4     
+ Misses        877      873       -4     

see 1 file with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

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.

thank you for joining as an owner for this component. 🙌

@pichlermarc pichlermarc merged commit 774dc42 into open-telemetry:main Apr 9, 2025
20 of 23 checks passed
@yiyuan-he yiyuan-he deleted the xray-sampler-owners branch April 9, 2025 15:58
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.

3 participants