-
Notifications
You must be signed in to change notification settings - Fork 70
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
Metrics tracing tags #1626
Metrics tracing tags #1626
Conversation
…ixed Bugs, improved UI for HeathState, Added Documentation
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## master #1626 +/- ##
============================================
+ Coverage 78.38% 78.59% +0.21%
- Complexity 2519 2579 +60
============================================
Files 257 266 +9
Lines 8257 8453 +196
Branches 984 995 +11
============================================
+ Hits 6472 6643 +171
- Misses 1358 1376 +18
- Partials 427 434 +7
|
Only the last four commits are relevant for this PR's feature. |
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.
Reviewed 1 of 9 files at r1, 8 of 23 files at r2, 7 of 255 files at r3, 212 of 240 files at r4, 9 of 10 files at r5, 33 of 35 files at r6, 5 of 5 files at r7, 7 of 7 files at r8, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @EddeCCC)
Include metrics tags also in tracing
This change is