Tinkoff / micro-sentry

Tiny Sentry client with idiomatic wrapper for Angular
Apache License 2.0
118 stars 8 forks source link

[BUG] Exclude tslib from bundle? #34

Closed danielcompton closed 2 years ago

danielcompton commented 2 years ago

🐞 Bug report

Description

https://bundlephobia.com/package/@micro-sentry/browser@6.0.0 shows that a big part of the package size comes from tslib. Is that necessary in the browser bundle?

Reproduction

Go to https://bundlephobia.com/package/@micro-sentry/browser@6.0.0

Expected behavior

https://bundlephobia.com/package/@micro-sentry/browser@6.0.0 doesn't show tslib (if possible?)

Versions

Additional context

It looks like tslib is only being used for __assign. Would it be possible to use a different polyfill or perhaps avoid Object.assign and add the keys directly?

tinkoff-bot commented 2 years ago

:tada: This issue is included in version core@6.0.2 :tada:

The release is available on GitHub release

tinkoff-bot commented 2 years ago

:tada: This issue is included in version browser@6.0.1 :tada:

The release is available on GitHub release

tinkoff-bot commented 2 years ago

:tada: This issue is included in version angular@6.1.4 :tada:

The release is available on GitHub release

danielcompton commented 2 years ago

Great, thanks! Looking good.

image

IKatsuba commented 2 years ago

@danielcompton thank you!