-
Notifications
You must be signed in to change notification settings - Fork 1
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
[PLATFORM-1473] Support opentelemetry 0.21 #154
Conversation
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.
I like the approach, because it gives us a way to release the latest version of the otel deps but lets users rollback on their own if it doesn't work
I think we should limit the supported versions and not end up like the actix library, but it's a matter of "discipline"
Is there a README
to update detailing this a bit? I.e. that we support two version and they can be used like so and blah blah
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.
One comment but I may be dumb
CHANGELOG.md
Outdated
### Added | ||
|
||
- Support for opentelemetry 0.21 | ||
- `tracing_opentelemetry_0_21` and `tracing_opentelemetry_0_21` features |
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.
I don't understand the difference 🤔
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.
Oops my bad, fixed
https://prima-assicurazioni-spa.myjetbrains.com/youtrack/issue/PLATFORM-1473/Support-opentelemetry-0.21