Skip to content

Comments

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

Merged
adinauer merged 38 commits into8.x.xfrom
feat/hsm-32-reuse-code-in-scopes
May 2, 2024
Merged

Hubs/Scopes Merge 32 - Reuse code in Scopes#3361
adinauer merged 38 commits into8.x.xfrom
feat/hsm-32-reuse-code-in-scopes

Conversation

@adinauer
Copy link
Member

@adinauer adinauer commented Apr 16, 2024

#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
Copy link
Contributor

github-actions bot commented Apr 16, 2024 •
edited
Loading

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

Generated by 🚫 dangerJS against 9255357

@github-actions
Copy link
Contributor

github-actions bot commented Apr 16, 2024 •
edited
Loading

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
@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

@stefanosiano stefanosiano Awaiting requested review from stefanosiano

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@adinauer @markushi @lbloder