Skip to content

ref(core): Extract _ensureBeforeSendRv into func #4288

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

Merged
merged 2 commits into from
Dec 15, 2021

Conversation

AbhiPrasad
Copy link
Member

@AbhiPrasad AbhiPrasad commented Dec 15, 2021

Extract out protected method _ensureBeforeSendRv from baseclient as it doesn't use any class props - this helps save a small amount of bundle size.

@github-actions
Copy link
Contributor

github-actions bot commented Dec 15, 2021

size-limit report

Path Base Size (1397a0a) Current Size Change
@sentry/browser - CDN Bundle (gzipped) 21.21 KB 21.21 KB -0.02% 🔽
@sentry/browser - CDN Bundle (minified) 68.58 KB 68.36 KB -0.32% 🔽
@sentry/browser - Webpack 22.92 KB 22.93 KB +0.07% 🔺
@sentry/browser - Webpack - gzip = false 79.49 KB 79.25 KB -0.31% 🔽
@sentry/react - Webpack 22.95 KB 22.97 KB +0.06% 🔺
@sentry/nextjs Client - Webpack 47.43 KB 47.43 KB -0.01% 🔽
@sentry/browser + @sentry/tracing - CDN Bundle (gzipped) 29.4 KB 29.41 KB +0.03% 🔺

@AbhiPrasad AbhiPrasad force-pushed the abhi-ensure-before-sendrv branch from 64cda8c to ff7a388 Compare December 15, 2021 13:58
@AbhiPrasad AbhiPrasad enabled auto-merge (squash) December 15, 2021 13:59
@AbhiPrasad AbhiPrasad merged commit 0a0887e into master Dec 15, 2021
@AbhiPrasad AbhiPrasad deleted the abhi-ensure-before-sendrv branch December 15, 2021 14:11
onurtemizkan pushed a commit that referenced this pull request Dec 19, 2021
Extract out protected method `_ensureBeforeSendRv` from `baseclient` as it doesn't use any class props - this helps save a small amount of bundle size.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants