-
Notifications
You must be signed in to change notification settings - Fork 810
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
docs(opencensus-shim) add an example showing how to use the opencensus shim #3778
Conversation
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## main #3778 +/- ##
==========================================
- Coverage 92.88% 92.47% -0.41%
==========================================
Files 297 165 -132
Lines 8836 3773 -5063
Branches 1814 757 -1057
==========================================
- Hits 8207 3489 -4718
+ Misses 629 284 -345 |
c65a10f
to
9d114e9
Compare
experimental/packages/opentelemetry-shim-opencensus/src/require-in-the-middle.d.ts
Outdated
Show resolved
Hide resolved
experimental/packages/opentelemetry-shim-opencensus/src/shim.ts
Outdated
Show resolved
Hide resolved
cff7d54
to
6b839e9
Compare
6b839e9
to
40b09f2
Compare
40b09f2
to
b55e74a
Compare
cd0cffa
to
c8212b9
Compare
1a65f04
to
b18ee45
Compare
b18ee45
to
418c4cc
Compare
418c4cc
to
4fca253
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.
This works great, thanks for adding the example 🙂
I took the liberty to update the @opentelemetry/*
to 1.14.0/0.40.0 to make the build pass. 🙂
Thank you Marc! Appreciate all the reviews |
…s shim (open-telemetry#3778) Co-authored-by: Marc Pichler <[email protected]>
Which problem is this PR solving?
Fixes #3749
Short description of the changes
Add an example app and update the README with some documentation
Type of change
Please delete options that are not relevant.
How Has This Been Tested?
Tested the example manually
Checklist: