-
Notifications
You must be signed in to change notification settings - Fork 21
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
Opentelemetry changes with optional tracing #178
Conversation
Hi @joe94 can you please review this? |
@utsavbatra5, taking a look at this today 👍 |
Hey @utsavbatra5, I applied some suggestions. Let me know if they look good to you 👍 |
Hi @joe thanks for reviewing and the commit. I have created a new commit for handling the failure of unit test cases. |
Codecov Report
@@ Coverage Diff @@
## main #178 +/- ##
==========================================
- Coverage 56.86% 54.91% -1.96%
==========================================
Files 16 16
Lines 932 967 +35
==========================================
+ Hits 530 531 +1
- Misses 392 424 +32
- Partials 10 12 +2
Continue to review full report at Codecov.
|
No description provided.