-
Notifications
You must be signed in to change notification settings - Fork 7
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Convert private span attribute to EmbraceAttributes #534
Conversation
This stack of pull requests is managed by Graphite. Learn more about stacking. Join @bidetofevil and the rest of your teammates on Graphite |
0f97f66
to
c5654b8
Compare
ae2f147
to
3da10fb
Compare
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
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #534 +/- ##
==========================================
+ Coverage 79.45% 79.51% +0.05%
==========================================
Files 403 408 +5
Lines 10861 10871 +10
Branches 1602 1602
==========================================
+ Hits 8630 8644 +14
+ Misses 1585 1581 -4
Partials 646 646
|
Merge activity
|
c5654b8
to
587bfd3
Compare
3da10fb
to
7a636ef
Compare
Goal
Convert
emb.private
to an EmbraceAttributeTesting
Existing tests cover this