-
-
Notifications
You must be signed in to change notification settings - Fork 1.7k
feat(browser): simplify wrap/fill #4286
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
size-limit report
|
I cannot find a real world situation where the
|
…script into feature/other-wrap
This is a small experiment what happens if we do not define
__sentry__
on our wrappers. It tries to reuse some code and removeshandleEvent
which is a dead code path in practice. I would prefer if i could also remove thewrap()
calls on the args. As far as I can see this is all undocumented behavior and it does not look like we depend on this.