Skip to content

Conversation

@trask
Copy link
Member

@trask trask commented Aug 19, 2025

Resolves #7571

I did add a test, but it didn't really seem worth keeping, let me know if you want it.

I did ask Copilot if there were any other similar issues in the repo, and FWIW it came back with:

The repository appears to be well-designed with respect to casting practices. The issue #7571 was an isolated incident where one method was inconsistent with the established pattern.

@trask trask marked this pull request as ready for review August 19, 2025 22:16
@trask trask requested a review from a team as a code owner August 19, 2025 22:16
@codecov
Copy link

codecov bot commented Aug 19, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 89.99%. Comparing base (20f0a3a) to head (d71ee62).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files
@@ Coverage Diff @@ ## main #7597 +/- ## ============================================ - Coverage 90.01% 89.99% -0.02%  + Complexity 7081 7080 -1  ============================================ Files 803 803 Lines 21419 21419 Branches 2086 2086 ============================================ - Hits 19280 19277 -3  - Misses 1477 1479 +2  - Partials 662 663 +1 

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
@jkwatson jkwatson merged commit f028e93 into open-telemetry:main Aug 20, 2025
29 checks passed
@trask trask deleted the cce branch October 20, 2025 17:27
the-clam pushed a commit to the-clam/opentelemetry-java that referenced this pull request Nov 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants