Skip to content
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.

Sign up for GitHub

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

Jump to bottom

ref(core): Refactor InboundFilters integration to use processEvent #9020

Merged
merged 1 commit into from
Sep 27, 2023
Merged

ref(core): Refactor InboundFilters integration to use processEvent #9020

merged 1 commit into from
Sep 27, 2023

Conversation

mydea
Copy link
Member

@mydea mydea commented Sep 13, 2023 •
edited
Loading

This refactors core integrations to use processEvent.

@mydea mydea self-assigned this Sep 13, 2023
@github-actions
Copy link
Contributor

github-actions bot commented Sep 13, 2023 •
edited
Loading

size-limit report 📦

Path Size
@sentry/browser (incl. Tracing, Replay) - Webpack (gzipped) 75.55 KB (-0.05% 🔽)
@sentry/browser (incl. Tracing) - Webpack (gzipped) 31.43 KB (-0.12% 🔽)
@sentry/browser - Webpack (gzipped) 22.03 KB (-0.17% 🔽)
@sentry/browser (incl. Tracing, Replay) - ES6 CDN Bundle (gzipped) 70.24 KB (-0.05% 🔽)
@sentry/browser (incl. Tracing) - ES6 CDN Bundle (gzipped) 28.54 KB (-0.13% 🔽)
@sentry/browser - ES6 CDN Bundle (gzipped) 20.61 KB (-0.2% 🔽)
@sentry/browser (incl. Tracing, Replay) - ES6 CDN Bundle (minified & uncompressed) 222.12 KB (-0.05% 🔽)
@sentry/browser (incl. Tracing) - ES6 CDN Bundle (minified & uncompressed) 86.48 KB (-0.11% 🔽)
@sentry/browser - ES6 CDN Bundle (minified & uncompressed) 61.33 KB (-0.15% 🔽)
@sentry/browser (incl. Tracing) - ES5 CDN Bundle (gzipped) 31.4 KB (-0.11% 🔽)
@sentry/react (incl. Tracing, Replay) - Webpack (gzipped) 75.57 KB (-0.05% 🔽)
@sentry/react - Webpack (gzipped) 22.05 KB (-0.17% 🔽)
@sentry/nextjs Client (incl. Tracing, Replay) - Webpack (gzipped) 93.46 KB (-0.03% 🔽)
@sentry/nextjs Client - Webpack (gzipped) 51.01 KB (-0.07% 🔽)

@mydea mydea force-pushed the fn/integration-processEvent branch from c2d603d to c354494 Compare September 14, 2023 14:42
Base automatically changed from fn/integration-processEvent to develop September 15, 2023 08:07
@mydea mydea force-pushed the fn/integrations-core branch 2 times, most recently from ff84d79 to 047adfc Compare September 21, 2023 07:23
@mydea mydea marked this pull request as ready for review September 21, 2023 07:23
@mydea mydea requested review from AbhiPrasad and lforst September 21, 2023 07:23
@mydea mydea force-pushed the fn/integrations-core branch from 047adfc to cf4174e Compare September 27, 2023 10:24
@mydea mydea merged commit 95e1801 into develop Sep 27, 2023
82 checks passed
@mydea mydea deleted the fn/integrations-core branch September 27, 2023 13:51
c298lee pushed a commit that referenced this pull request Sep 27, 2023
@mydea @c298lee
ref(core): Refactor InboundFilters integration to use processEvent (
a753bc4 
#9020)

This refactors core integrations to use `processEvent`.
@lforst lforst mentioned this pull request Sep 29, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@AbhiPrasad AbhiPrasad AbhiPrasad approved these changes

@lforst lforst Awaiting requested review from lforst

Assignees

@mydea mydea

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@mydea @AbhiPrasad