Skip to content

Small refactoring to make safety clearer#8083

Merged
jack-berg merged 1 commit intoopen-telemetry:mainfrom
trask:safe
Feb 13, 2026
Merged

Small refactoring to make safety clearer#8083
jack-berg merged 1 commit intoopen-telemetry:mainfrom
trask:safe

Conversation

@trask
Copy link
Member

@trask trask commented Feb 13, 2026

Alternatively, I can just add comment explaining that the safety relies on the caller not reusing the byte buffer

@trask trask requested a review from a team as a code owner February 13, 2026 18:38
@codecov
Copy link

codecov bot commented Feb 13, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 90.22%. Comparing base (673f89e) to head (1b9630f).
⚠️ Report is 3 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff            @@
##               main    #8083   +/-   ##
=========================================
  Coverage     90.22%   90.22%           
+ Complexity     7606     7605    -1     
=========================================
  Files           841      841           
  Lines         22938    22939    +1     
  Branches       2291     2291           
=========================================
+ Hits          20695    20697    +2     
+ Misses         1526     1524    -2     
- Partials        717      718    +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.

@jack-berg jack-berg merged commit cfe93d0 into open-telemetry:main Feb 13, 2026
27 checks passed
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.

2 participants