-
-
Notifications
You must be signed in to change notification settings - Fork 1.7k
[Gitflow] Merge develop into master #7111
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
Merged
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
[Gitflow] Merge master into develop
This has been bothering me because `yarn install` on an M1 fails due to puppeteer. Note: as of current v19 is latest, but it's incompatible with our version of TS (3.9)
Add a few Playwright integration test helper functions for Replay integration tests Add tests for some custom events we add to the rrweb recording (breadcrumbs and performance spans)
Co-authored-by: Luca Forstner <[email protected]>
Based on getsentry/rrweb#41, we can now ignore rrweb errors even when the code is minified.
Add Playwright integration test utilities to test Replay's zlib compression. We now have a dedicated parser that attempts to decompress a payload, if it encounters a zlib-compressed replay recording payload. Additionally, add a simple test to check that an initial full compressed snapshot has the correct content.
The previous PR that introduces the Replay SDK metrics job already contained code that collected baseline info from the main branch. However, after making a change to only run when a specific label is added, this feature got lost... This PR should fix that.
Oops, nested the `pull_request` inside of the `push`!
This updates rrweb to https://github.com/getsentry/rrweb/releases/tag/1.101.0, which mainly improves our wrapping of rrweb errors.
Co-authored-by: Lukas Stracke <[email protected]>
Co-authored-by: Abhijeet Prasad <[email protected]>
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.
Auto approved automated PR
size-limit report 📦
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Merge develop branch into master