Skip to content

feat(replay): Remove unused debug statements - #8167

Merged
billyvg merged 2 commits into
developfrom
feat-replay-remove-debug-statement
May 25, 2023
Merged

feat(replay): Remove unused debug statements#8167
billyvg merged 2 commits into
developfrom
feat-replay-remove-debug-statement

Conversation

@billyvg

@billyvg billyvg commented May 19, 2023

Copy link
Copy Markdown
Member

These debug statements are not really providing value, removing them...

These debug statements are not really providing value, removing them...
@github-actions

github-actions Bot commented May 19, 2023

Copy link
Copy Markdown
Contributor

size-limit report 📦

Path Size
@sentry/browser - ES5 CDN Bundle (gzipped + minified) 21.03 KB (+0.06% 🔺)
@sentry/browser - ES5 CDN Bundle (minified) 65.68 KB (+0.09% 🔺)
@sentry/browser - ES6 CDN Bundle (gzipped + minified) 19.59 KB (+0.11% 🔺)
@sentry/browser - ES6 CDN Bundle (minified) 58.15 KB (+0.12% 🔺)
@sentry/browser - Webpack (gzipped + minified) 21.2 KB (+0.15% 🔺)
@sentry/browser - Webpack (minified) 69.07 KB (+0.05% 🔺)
@sentry/react - Webpack (gzipped + minified) 21.22 KB (+0.16% 🔺)
@sentry/nextjs Client - Webpack (gzipped + minified) 49.16 KB (+0.09% 🔺)
@sentry/browser + @sentry/tracing - ES5 CDN Bundle (gzipped + minified) 28.67 KB (+0.09% 🔺)
@sentry/browser + @sentry/tracing - ES6 CDN Bundle (gzipped + minified) 26.9 KB (+0.1% 🔺)
@sentry/replay ES6 CDN Bundle (gzipped + minified) 48.24 KB (+1.32% 🔺)
@sentry/replay - Webpack (gzipped + minified) 41.95 KB (+1.19% 🔺)
@sentry/browser + @sentry/tracing + @sentry/replay - ES6 CDN Bundle (gzipped + minified) 67.21 KB (+1.07% 🔺)
@sentry/browser + @sentry/replay - ES6 CDN Bundle (gzipped + minified) 60.11 KB (+1.23% 🔺)

@billyvg
billyvg marked this pull request as ready for review May 23, 2023 18:05
@billyvg
billyvg requested a review from Lms24 May 24, 2023 17:20
@billyvg
billyvg merged commit 3f31b3d into develop May 25, 2023
@billyvg
billyvg deleted the feat-replay-remove-debug-statement branch May 25, 2023 12:48
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