-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Split stdout exporter into stdouttrace and stdoutmetric #2005
Conversation
Signed-off-by: Anthony J Mirabella <[email protected]>
a5507c4
to
d8faeb9
Compare
Codecov Report
@@ Coverage Diff @@
## main #2005 +/- ##
=====================================
Coverage 72.8% 72.8%
=====================================
Files 169 171 +2
Lines 8667 8675 +8
=====================================
+ Hits 6311 6321 +10
+ Misses 2084 2082 -2
Partials 272 272
|
Signed-off-by: Anthony J Mirabella <[email protected]>
Signed-off-by: Anthony J Mirabella <[email protected]>
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.
I don't know how deep the Without* options run. It would be nice to have them out but I would be ok with leaving them, as this is intended to be a tool for us to develop with.
Thanks for putting this together.
…o fix/stdout_exporter
…er scope Signed-off-by: Anthony J Mirabella <[email protected]>
…signal exporters Signed-off-by: Anthony J Mirabella <[email protected]>
Signed-off-by: Anthony J Mirabella [email protected]