Skip to content

ref(deno): Remove @sentry/browser dependency from @sentry/deno#10867

Merged
s1gr1d merged 2 commits intodevelopfrom
sig-delete-browser-dep-in-deno
Mar 1, 2024
Merged

ref(deno): Remove @sentry/browser dependency from @sentry/deno#10867
s1gr1d merged 2 commits intodevelopfrom
sig-delete-browser-dep-in-deno

Conversation

@s1gr1d
Copy link
Copy Markdown
Member

@s1gr1d s1gr1d commented Feb 29, 2024

Adds the breadcrumbsIntegration (without xhr, dom, history) to deno without importing that from browser.

@s1gr1d s1gr1d requested a review from mydea February 29, 2024 15:52
@AbhiPrasad AbhiPrasad requested a review from timfish February 29, 2024 15:54
* This breadcrumbsIntegration is almost the same as the one from @sentry/browser.
* The Deno-version does not support browser-specific APIs like dom, xhr and history.
*/
const _breadcrumbsIntegration = ((options: Partial<BreadcrumbsOptions> = {}) => {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

l: Let's move the docstring from here to export const breadcrumbsIntegration so it shows up in user's IDEs.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Feb 29, 2024

size-limit report 📦

Path Size
@sentry/browser (incl. Tracing, Replay, Feedback) - Webpack (gzipped) 77.4 KB (0%)
@sentry/browser (incl. Tracing, Replay) - Webpack (gzipped) 68.61 KB (0%)
@sentry/browser (incl. Tracing, Replay with Canvas) - Webpack (gzipped) 72.54 KB (0%)
@sentry/browser (incl. Tracing, Replay) - Webpack with treeshaking flags (gzipped) 62.16 KB (0%)
@sentry/browser (incl. Tracing) - Webpack (gzipped) 32.84 KB (0%)
@sentry/browser (incl. browserTracingIntegration) - Webpack (gzipped) 32.84 KB (0%)
@sentry/browser (incl. Feedback) - Webpack (gzipped) 31.06 KB (0%)
@sentry/browser (incl. sendFeedback) - Webpack (gzipped) 31.06 KB (0%)
@sentry/browser - Webpack (gzipped) 22.3 KB (0%)
@sentry/browser (incl. Tracing, Replay, Feedback) - ES6 CDN Bundle (gzipped) 75.83 KB (0%)
@sentry/browser (incl. Tracing, Replay) - ES6 CDN Bundle (gzipped) 67.45 KB (0%)
@sentry/browser (incl. Tracing) - ES6 CDN Bundle (gzipped) 33.28 KB (0%)
@sentry/browser - ES6 CDN Bundle (gzipped) 24.79 KB (0%)
@sentry/browser (incl. Tracing, Replay) - ES6 CDN Bundle (minified & uncompressed) 211.08 KB (0%)
@sentry/browser (incl. Tracing) - ES6 CDN Bundle (minified & uncompressed) 99.81 KB (0%)
@sentry/browser - ES6 CDN Bundle (minified & uncompressed) 74.07 KB (0%)
@sentry/browser (incl. Tracing) - ES5 CDN Bundle (gzipped) 36.35 KB (0%)
@sentry/react (incl. Tracing, Replay) - Webpack (gzipped) 68.88 KB (-0.01% 🔽)
@sentry/react - Webpack (gzipped) 22.33 KB (0%)
@sentry/nextjs Client (incl. Tracing, Replay) - Webpack (gzipped) 85.36 KB (0%)
@sentry/nextjs Client - Webpack (gzipped) 49.69 KB (0%)
@sentry-internal/feedback - Webpack (gzipped) 17.09 KB (0%)

@s1gr1d s1gr1d self-assigned this Mar 1, 2024
@s1gr1d s1gr1d force-pushed the sig-delete-browser-dep-in-deno branch from e6fc01f to 05b1276 Compare March 1, 2024 09:24
@s1gr1d s1gr1d merged commit 30cadf9 into develop Mar 1, 2024
@s1gr1d s1gr1d deleted the sig-delete-browser-dep-in-deno branch March 1, 2024 09:39
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