Skip to content

feat(sveltekit): Remove withSentryViteConfig#7789

Merged
Lms24 merged 1 commit intodevelopfrom
lms/sveltekit-remove-withsentryviteconfig
Apr 7, 2023
Merged

feat(sveltekit): Remove withSentryViteConfig#7789
Lms24 merged 1 commit intodevelopfrom
lms/sveltekit-remove-withsentryviteconfig

Conversation

@Lms24
Copy link
Copy Markdown
Member

@Lms24 Lms24 commented Apr 7, 2023

This PR removes the withSentryViteConfig wrapper in favour of sentrySvelteKitPlugin introduced in #7788.

ref #7787

@Lms24 Lms24 requested review from AbhiPrasad and lforst April 7, 2023 13:33
@Lms24 Lms24 self-assigned this Apr 7, 2023
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 7, 2023

size-limit report 📦

Path Size
@sentry/browser - ES5 CDN Bundle (gzipped + minified) 20.94 KB (+0.08% 🔺)
@sentry/browser - ES5 CDN Bundle (minified) 65.44 KB (+0.03% 🔺)
@sentry/browser - ES6 CDN Bundle (gzipped + minified) 19.49 KB (+0.08% 🔺)
@sentry/browser - ES6 CDN Bundle (minified) 57.88 KB (+0.04% 🔺)
@sentry/browser - Webpack (gzipped + minified) 21.11 KB (+0.06% 🔺)
@sentry/browser - Webpack (minified) 68.87 KB (+0.03% 🔺)
@sentry/react - Webpack (gzipped + minified) 21.14 KB (+0.07% 🔺)
@sentry/nextjs Client - Webpack (gzipped + minified) 48.9 KB (+0.03% 🔺)
@sentry/browser + @sentry/tracing - ES5 CDN Bundle (gzipped + minified) 28.5 KB (+0.06% 🔺)
@sentry/browser + @sentry/tracing - ES6 CDN Bundle (gzipped + minified) 26.73 KB (+0.06% 🔺)
@sentry/replay ES6 CDN Bundle (gzipped + minified) 44.89 KB (-0.07% 🔽)
@sentry/replay - Webpack (gzipped + minified) 38.85 KB (-0.06% 🔽)
@sentry/browser + @sentry/tracing + @sentry/replay - ES6 CDN Bundle (gzipped + minified) 63.78 KB (-0.05% 🔽)
@sentry/browser + @sentry/replay - ES6 CDN Bundle (gzipped + minified) 56.82 KB (-0.05% 🔽)

@Lms24 Lms24 merged commit d099af8 into develop Apr 7, 2023
@Lms24 Lms24 deleted the lms/sveltekit-remove-withsentryviteconfig branch April 7, 2023 13:54
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