-
-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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.
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
feat: SDK unified API #1402
feat: SDK unified API #1402
Conversation
packages/hub/src/hub.ts
Outdated
*/ | ||
public addEventProcessor(callback: () => (event: SentryEvent) => void): void { | ||
const top = this.getStackTop(); | ||
if (top.scope) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should be top.client
.
packages/hub/src/global.ts
Outdated
* | ||
* If a hub is already registered in the global carrier but this module | ||
* contains a more recent version, it replaces the registered version. | ||
* Otherwise, the currently registered hub will be returned. | ||
*/ | ||
export function getGlobalHub(): Hub { | ||
const registry = getGlobalCarrier(); | ||
export function getMainHub(): Hub { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If it returns domain local data it should be getDefaultHub
.
Codecov Report
@@ Coverage Diff @@
## master #1402 +/- ##
==========================================
+ Coverage 88.09% 88.73% +0.63%
==========================================
Files 31 31
Lines 697 701 +4
Branches 105 105
==========================================
+ Hits 614 622 +8
+ Misses 78 74 -4
Partials 5 5
Continue to review full report at Codecov.
|
This PR makes the SDK unification paper conform