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.

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

Bump: JavaScript 7.93.0 #573

Merged
merged 18 commits into from
Mar 5, 2024
Merged

Bump: JavaScript 7.93.0 #573

merged 18 commits into from
Mar 5, 2024

Conversation

lucas-zimerman
Copy link
Collaborator

@lucas-zimerman lucas-zimerman commented Feb 15, 2024

This PR Bumps Sentry JavaScript to version 7.93
There are internal break changes that required some code change but it shouldn't have any impact on the user side.
Before approval, this PR depends on #572 to fix the rewriteframes tests.

@lucas-zimerman lucas-zimerman changed the title Bump: JavaScript 7.100.1 Bump: JavaScript 7.93.0 Feb 15, 2024
@lucas-zimerman lucas-zimerman marked this pull request as ready for review February 23, 2024 00:08
CHANGELOG.md Outdated Show resolved Hide resolved
test/testutils.ts Outdated Show resolved Hide resolved
lucas-zimerman and others added 3 commits February 23, 2024 12:03
* Add tests, fix errors and enhance sample

* fix working path

* test

* remove working directory

* add sample

* add test

* test

* fix sample path

* ignore cli errors

* fix hybridsdk

* update lock

* wrong slash

* update samples

* small fixes

* test

* test

* missing esm

* test

* oops

* removed unused Native Linked Errors integration, also fixed missing vue build script, removed tests for the removed code

* Update test/integrations/rewriteframes.test.ts

Co-authored-by: Kryštof Woldřich <31292499+krystofwoldrich@users.noreply.github.com>

* Update test/integrations/rewriteframes.test.ts

Co-authored-by: Kryštof Woldřich <31292499+krystofwoldrich@users.noreply.github.com>

* remove client

* Add capacitor production tests

* removed unrelated tests

* add development test

* Update src/integrations/rewriteframes.ts

---------

Co-authored-by: Kryštof Woldřich <31292499+krystofwoldrich@users.noreply.github.com>
Co-authored-by: Kryštof Woldřich <31292499+krystofwoldrich@users.noreply.github.com>
Copy link
Member

@krystofwoldrich krystofwoldrich left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 🚀

I have only one nit about the eventorigin integration callback name, I'll leave that up to you.

@lucas-zimerman lucas-zimerman merged commit ebec8e4 into main Mar 5, 2024
7 checks passed
@lucas-zimerman lucas-zimerman deleted the bump/7.100.1 branch March 5, 2024 11:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants