davidbarsky / tracing-tree

Apache License 2.0
123 stars 31 forks source link

Upgrade to tracing-subscriber 0.3 #36

Closed theduke closed 2 years ago

theduke commented 2 years ago

Tracing-subscriber 0.3 was released, would be great to see an upgrade here as well.

davidbarsky commented 2 years ago

yeah, i agree. tracing-tree uses a deprecated method in tracing-subscriber and their replacements seem to require foreknowledge as to whether the scope is an event or a span—otherwise, the formatting looks off. i haven't had the time (yet) to fix this.

davidbarsky commented 2 years ago

This was resolved in #40. I'll have a 0.2 release with those changes out shortly.

theduke commented 2 years ago

Great, thanks!