Skip to content

Conversation

bogdandrutu
Copy link
Member

Small issue that the SpanNameSep is duplicated, but is a very small duplicate.

@bogdandrutu bogdandrutu requested review from a team and djaglowski September 11, 2024 19:59
@bogdandrutu bogdandrutu added the Skip Changelog PRs that do not require a CHANGELOG.md entry label Sep 11, 2024
Copy link

codecov bot commented Sep 11, 2024

Codecov Report

Attention: Patch coverage is 94.73684% with 2 lines in your changes missing coverage. Please review.

Project coverage is 92.26%. Comparing base (168bba7) to head (3b2f460).
Report is 5 commits behind head on main.

Files with missing lines Patch % Lines
processor/processorhelper/obsreport.go 60.00% 1 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main   #11148      +/-   ##
==========================================
- Coverage   92.28%   92.26%   -0.02%     
==========================================
  Files         414      414              
  Lines       19798    19802       +4     
==========================================
  Hits        18270    18270              
- Misses       1156     1160       +4     
  Partials      372      372              

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

@bogdandrutu bogdandrutu merged commit 1245955 into open-telemetry:main Sep 11, 2024
48 of 49 checks passed
@github-actions github-actions bot added this to the next release milestone Sep 11, 2024
@bogdandrutu bogdandrutu deleted the rm-obsreportconfig branch September 11, 2024 21:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Skip Changelog PRs that do not require a CHANGELOG.md entry

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants