search

typelevel / cats-effect

The pure asynchronous runtime for Scala
https://typelevel.org/cats-effect/
Apache License 2.0
2.01k stars 515 forks source link

Collapse `tracing` package into `cats.effect` to avoid bloating the FFI #2524

Open djspiewak opened 2 years ago

djspiewak commented 2 years ago

The fact that tracing is a separate package means that TracingConstants needs to be public. We can avoid that problem if we squeeze it into the cats.effect package, along with the other implementation details.

vasilmkd commented 2 years ago

We now cannot use class TracingConstants in the unsafe package. This is frustrating.

vasilmkd commented 2 years ago

At this points, it's either public class or code duplication.

armanbilge commented 2 years ago

Or, becoming a flat-earther 😆