Deprecate opentracing shim public API#8373
Conversation
|
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #8373 +/- ##
=========================================
Coverage 90.88% 90.88%
Complexity 7985 7985
=========================================
Files 898 898
Lines 24107 24107
Branches 2406 2406
=========================================
Hits 21909 21909
Misses 1455 1455
Partials 743 743 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
jack-berg
left a comment
There was a problem hiding this comment.
Looks good. Merge latest main, resolve conflicts, revert the unrelated graalvm configuration cache changes and I'll get this merged. Thanks!
| notCompatibleWithConfigurationCache( | ||
| "GraalVM native test agent options are not serializable for this task.", | ||
| ) | ||
| } |
|
Merged latest main, resolved the conflicts, reverted the unrelated graal-incubating configuration-cache change, and updated the opentracing-shim apidiff against the current baseline. The branch has been pushed and checks are rerunning now. |
|
@jack-berg The remaining |
Deprecate the public
opentracing-shimentrypoints and update the module documentation in line with the OpenTelemetry specification guidance on OpenTracing compatibility.This keeps behavior unchanged and limits the scope to deprecation annotations and docs for the legacy shim surface.
Fixes #8323