-
-
Notifications
You must be signed in to change notification settings - Fork 1.7k
feat(core)!: Remove BaseClient
#17071
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
Conversation
size-limit report 📦
|
] as Envelope; | ||
it('should call a beforeEnvelope hook', () => { | ||
const client = new TestClient(options); | ||
expect.assertions(1); |
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.
I know 100% this is a Andrei PR when I see this :D
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.
it already existed 😎
`BaseClient` was deprecated in v9, use `Client` as a direct replacement.
8f905be
to
8270e15
Compare
* chore(ci): v7: Handle GH runner image deprecations * chore(deps): v7: update Android SDK to v8.20.0 * Adds changelog * Set specific xcode version for expo (see expo/expo#36830) * Bump Javascript SDK to 9.44.2 * Bump Javascript SDK to 10.4.0 and rename deprecated base client getsentry/sentry-javascript#17071 * Renames logger getsentry/sentry-javascript#16901 * Bump Javascript SDK to 10.7.0 * Adds changelog * Sets skipLibCheck to true in ts3.8 typecheck * Fixes fake timers jest issue after getsentry/sentry-javascript#16890 * Fixes lint issue * Apply suggestions from code review Co-authored-by: LucasZF <lucas-zimerman1@hotmail.com> * Update yarn.lock --------- Co-authored-by: LucasZF <lucas-zimerman1@hotmail.com>
BaseClient
was deprecated in v9, useClient
as a direct replacement.