Skip to content

Add http semantic conventions notice to changelog#6737

Merged
dmathieu merged 7 commits into
mainfrom
semconv-notice
Feb 20, 2025
Merged

Add http semantic conventions notice to changelog#6737
dmathieu merged 7 commits into
mainfrom
semconv-notice

Conversation

@dmathieu
Copy link
Copy Markdown
Member

@dmathieu dmathieu commented Feb 7, 2025

Closes #6731

@dmathieu dmathieu added this to the v1.35.0 milestone Feb 7, 2025
@dmathieu dmathieu marked this pull request as ready for review February 7, 2025 09:09
@dmathieu dmathieu requested a review from a team as a code owner February 7, 2025 09:09
@dmathieu
Copy link
Copy Markdown
Member Author

dmathieu commented Feb 7, 2025

We should wait for #6710 to be merged before merging this one.

Comment thread CHANGELOG.md Outdated
Comment thread CHANGELOG.md Outdated
@dmathieu dmathieu requested review from MrAlias and pellared February 13, 2025 10:46
@dmathieu
Copy link
Copy Markdown
Member Author

@open-telemetry/go-approvers this needs an approval from someone not working at splunk.

@codecov
Copy link
Copy Markdown

codecov Bot commented Feb 20, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 75.5%. Comparing base (6da8e3c) to head (31092c8).
Report is 1 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@          Coverage Diff          @@
##            main   #6737   +/-   ##
=====================================
  Coverage   75.4%   75.5%           
=====================================
  Files        206     206           
  Lines      19181   19181           
=====================================
+ Hits       14481   14484    +3     
+ Misses      4261    4259    -2     
+ Partials     439     438    -1     

see 1 file with indirect coverage changes

@dmathieu dmathieu merged commit 39ff1c5 into main Feb 20, 2025
@dmathieu dmathieu deleted the semconv-notice branch February 20, 2025 17:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Development

Successfully merging this pull request may close these issues.

Add a warning to the changelog for the upcoming HTTP migration

4 participants