-
-
Notifications
You must be signed in to change notification settings - Fork 1.7k
feat(v8): Remove extractTraceparentData export
#10559
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
Conversation
0639a2e to
43a3b87
Compare
size-limit report 📦
|
Lms24
left a comment
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.
Is there a reason why we no longer export this API in the top level packages?
|
I think we prefer people import it from utils (which will live in For now let's move forward with removing it as promised with the deprecation. We can always figure out how to bring it back in an alternate fashion. |
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.
Right, makes sense, thx!
43a3b87 to
97cf69e
Compare
Instead, import this directly from
@sentry/utils.Should help when we move
@sentry/utilsinto core.