cats-effect icon indicating copy to clipboard operation
cats-effect copied to clipboard

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

Open djspiewak opened this issue 4 years ago • 3 comments

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.

djspiewak avatar Nov 09 '21 06:11 djspiewak

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

vasilmkd avatar Nov 09 '21 08:11 vasilmkd

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

vasilmkd avatar Nov 09 '21 08:11 vasilmkd

Or, becoming a flat-earther 😆

armanbilge avatar Nov 09 '21 08:11 armanbilge