-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Remove support for deprecated unmarshaler #2591
Remove support for deprecated unmarshaler #2591
Conversation
Updates open-telemetry#2540 Signed-off-by: Bogdan Drutu <[email protected]>
d9f899c
to
57d8f01
Compare
Codecov Report
@@ Coverage Diff @@
## main #2591 +/- ##
==========================================
- Coverage 92.03% 92.03% -0.01%
==========================================
Files 273 273
Lines 15455 15449 -6
==========================================
- Hits 14224 14218 -6
Misses 849 849
Partials 382 382
Continue to review full report at Codecov.
|
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.
Are there any contrib components that still use deprecated CustomUnmarshaler or all are fixed now?
@tigrannajaryan all are fixed and they are switched also to the helper "WithCustomUnmarshaler" from components helpers |
Updates #2540