-
Notifications
You must be signed in to change notification settings - Fork 562
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
otellogr: Implement Info, WithName, WithValues #6111
otellogr: Implement Info, WithName, WithValues #6111
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #6111 +/- ##
=======================================
+ Coverage 66.4% 66.8% +0.3%
=======================================
Files 205 206 +1
Lines 13093 13207 +114
=======================================
+ Hits 8706 8829 +123
+ Misses 4119 4111 -8
+ Partials 268 267 -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.
Just two comments left. Nice work 👍
Co-authored-by: Robert Pająk <[email protected]>
Co-authored-by: Robert Pająk <[email protected]>
@dmathieu, can you please take a look? |
This PR implements a few methods for otellogr package
Out of scope
Please add skip changelog thx 🙏
Part of #5192