-
Notifications
You must be signed in to change notification settings - Fork 44
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
chore: Update opentelemetry
dependencies
#1114
Conversation
I'm currently unfamiliar with the actual usage of |
Benchmark ResultsSummary
✅ See Better Results...
❌ See Worse Results...
✨ See Unchanged Results...
🐋 See Full Results... |
Codecov Report
@@ Coverage Diff @@
## develop #1114 +/- ##
===========================================
+ Coverage 67.73% 67.74% +0.01%
===========================================
Files 181 181
Lines 16551 16551
===========================================
+ Hits 11210 11212 +2
+ Misses 4400 4399 -1
+ Partials 941 940 -1
|
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.
Thanks for the change Orpheus. I had done it in my PR but then reverted cuz I wasn't sure if Shahzad wanted to make the change yet but I think it makes sense. It's a pretty straight forward change.
Sorry for the late response @orpheuslummis thought would still let you know it all looks okay 👍 |
Great. Sorry I forgot you were taking some time off! |
Relevant issue(s)
Resolves #1113
Last issue of #317 then we can do #320
Description
Update to latest opentelemetry dependencies. It includes handling of breaking changes.
At this point in time, this brings us to having all most-recent direct dependencies.
Tasks
How has this been tested?
make test
Specify the platform(s) on which this was tested: