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

Hubs/Scopes Merge 32 - Reuse code in Scopes #3361

Merged
merged 38 commits into from
May 2, 2024
Merged

Hubs/Scopes Merge 32 - Reuse code in Scopes #3361

merged 38 commits into from
May 2, 2024

Conversation

adinauer
Copy link
Member

#skip-changelog

📜 Description

💡 Motivation and Context

💚 How did you test it?

📝 Checklist

  • I reviewed the submitted code.
  • I added tests to verify the changes.
  • No new PII added or SDK only sends newly added PII if sendDefaultPII is enabled.
  • I updated the docs if needed.
  • Review from the native team if needed.
  • No breaking change or entry added to the changelog.
  • No breaking change for hybrid SDKs or communicated to hybrid SDKs.

🔮 Next steps

adinauer and others added 30 commits March 27, 2024 16:30
@adinauer
replace hub with scopes
305baf5
@adinauer
Add Scopes
95f5e1b
@adinauer
Introduce IScopes interface.
27f2398
@adinauer
Replace IHub with IScopes in core
ce3c14f
@adinauer
Replace IHub with IScopes in android core
ce615f4
@adinauer
Replace IHub with IScopes in android integrations
22ddc00
@adinauer
Replace IHub with IScopes in apollo integrations
305c217
@adinauer
Replace IHub with IScopes in okhttp integration
da927bc
@adinauer
Replace IHub with IScopes in graphql integration
8279276
@adinauer
Replace IHub with IScopes in logging integrations
9bfc086
@adinauer
Replace IHub with IScopes in more integrations
b998e50
@adinauer
Replace IHub with IScopes in OTel integration
739827a
@adinauer
Replace IHub with IScopes in Spring 5 / Spring Boot 2 integrations
69f2d63
@adinauer
Replace IHub with IScopes in Spring 6 / Spring Boot 3 integrations
792d482
@adinauer
Replace IHub with IScopes in samples
9bcbce6
@adinauer
Merge branch 'feat/hsm-13-replacements-in-samples' into feat/hubs-sco…
3f25a4b 
…pes-merge-2-add-scopes
@adinauer
gitscopes -> github
d6fb40a
@adinauer
Replace ThreadLocal with ScopesStorage
7752bcc
@adinauer
Move client and throwable to span map to scope
1e329c5
@adinauer
Add global scope
b0d89ae
@adinauer
use global scope in Scopes
cdd414a
@adinauer
Implement pushScope popScope and withScope for Scopes
98da9ff
@adinauer
Add pushIsolationScope; add fork methods to ISCope
2d26033
@adinauer
Use separate scopes for current, isolation and global scope; rename m…
bbb6700 
…ainScopes to rootScopes
@adinauer
Allow controlling which scope configureScope uses
c714b21
@adinauer
Combine scopes
a474402
@adinauer
Use new API for CRONS integrations
ae93e33
@adinauer
Add lifecycle helper
b01298b
@adinauer
Change spring integrations to use new API
b64e688
@adinauer
Use new API in servlet integrations
d06fc50
@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Apr 16, 2024 •
edited

Messages
📖 Do not forget to update Sentry-docs with your feature once the pull request gets approved.

Generated by 🚫 dangerJS against 9255357

@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Apr 16, 2024 •
edited

Performance metrics 🚀

  Plain With Sentry Diff
Startup time 330.94 ms 379.06 ms 48.13 ms
Size 1.70 MiB 2.28 MiB 596.13 KiB

@adinauer adinauer mentioned this pull request Apr 16, 2024
Merged
7 tasks
Base automatically changed from feat/hsm-31-event-processor-ordering to 8.x.x May 2, 2024 12:04
@adinauer adinauer merged commit 10f8e44 into 8.x.x May 2, 2024
18 of 19 checks passed
@adinauer adinauer deleted the feat/hsm-32-reuse-code-in-scopes branch May 2, 2024 12:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@markushi markushi markushi approved these changes

@lbloder lbloder lbloder approved these changes

@romtsn romtsn Awaiting requested review from romtsn romtsn is a code owner

@stefanosiano stefanosiano Awaiting requested review from stefanosiano stefanosiano is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@adinauer @markushi @lbloder