-
Notifications
You must be signed in to change notification settings - Fork 47
Conversation
Codecov Report
@@ Coverage Diff @@
## main #142 +/- ##
=======================================
Coverage 94.46% 94.47%
=======================================
Files 42 42
Lines 578 579 +1
Branches 94 94
=======================================
+ Hits 546 547 +1
Misses 32 32
Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Since we are marking SpanAttributes
as deprecated, should we have a follow up task to replace usage with Attributes
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Since there are already 3 approvals, mark the PR as changes requested to prevent accidental merging until we solve the questions.
As discussed in SIG, this creates a common definition for attributes which can be used by all signals.
Span attributes is marked as deprecated in favor of the new common attributes definition.