Skip to content

Conversation

@aabmass
Copy link
Member

@aabmass aabmass commented Aug 29, 2025

Which problem is this PR solving?

Keep component owners fresh. Punya is no longer working on this code.

cc @psx95, @DylanRussell, @dashpole

Short description of the changes

N/A

@aabmass aabmass requested a review from a team as a code owner August 29, 2025 22:41
@aabmass aabmass changed the title chore(resource-detector-gcp): replace punya with jsuereth as componen… chore(resource-detector-gcp): replace punya with jsuereth as component owner Aug 29, 2025
@codecov
Copy link

codecov bot commented Aug 29, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 97.62%. Comparing base (c1866e1) to head (e8a9078).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3006      +/-   ##
==========================================
+ Coverage   89.58%   97.62%   +8.03%     
==========================================
  Files         193        9     -184     
  Lines        9729      505    -9224     
  Branches     2025       89    -1936     
==========================================
- Hits         8716      493    -8223     
+ Misses       1013       12    -1001     

see 184 files 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.

@aabmass
Copy link
Member Author

aabmass commented Sep 2, 2025

Discussed with coworkers and I'm updating this PR to include @psx95 @DylanRussell and @dashpole

@aabmass aabmass changed the title chore(resource-detector-gcp): replace punya with jsuereth as component owner chore(resource-detector-gcp): update component owners for GCP detector Sep 2, 2025
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.

Thanks @aabmass! 🙂
@DylanRussell and @dashpole please approve this PR to indicate that you're okay with this ✅

@pichlermarc pichlermarc enabled auto-merge (squash) September 3, 2025 15:43
@pichlermarc pichlermarc merged commit 511e5fd into open-telemetry:main Sep 3, 2025
21 checks passed
@aabmass aabmass deleted the remove-punya-owner branch September 3, 2025 16:16
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.

5 participants