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

[v8] Merge @sentry/integrations into @sentry/core #9833

Closed
Tracked by #9508
mydea opened this issue Dec 14, 2023 · 1 comment
Closed
Tracked by #9508

[v8] Merge @sentry/integrations into @sentry/core #9833

mydea opened this issue Dec 14, 2023 · 1 comment
Assignees
@s1gr1d
Milestone
8.0.0

Comments

@mydea
Copy link
Member

mydea commented Dec 14, 2023

In v8, let's just export the pluggable integrations directly from @sentry/core (and all downstream packages). These can be tree shaken. Runtime-specific integrations should be moved to browser/node/server-runtime packages instead.
@mydea mydea added this to the 8.0.0 milestone Dec 14, 2023
@mydea mydea mentioned this issue Dec 14, 2023
Closed
@AbhiPrasad AbhiPrasad mentioned this issue Feb 7, 2024
Merged
@mydea
Copy link
Member Author

mydea commented Feb 21, 2024

HttpClient, ContextLines, ReportingObserver should go into browser, the rest into core I believe!

@s1gr1d s1gr1d mentioned this issue Feb 23, 2024
Merged
s1gr1d added a commit that referenced this issue Feb 27, 2024
@s1gr1d
feat(v8/integrations): Merge integrations into core (#10799)
a5c4fcb 
Move relevant integrations. Part of
#9833
@s1gr1d s1gr1d closed this as completed Feb 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@s1gr1d s1gr1d

Labels
None yet
Projects
None yet
Milestone
8.0.0
Development

No branches or pull requests
2 participants
@mydea @s1gr1d