-
Notifications
You must be signed in to change notification settings - Fork 142
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
v6-beta #2883
v6-beta #2883
Conversation
Bundles Sizes Evolution
🚀 CPU Performance
🧠 Memory Performance
|
/to-staging |
🚂 Branch Integration: starting soon, median merge time is 12m Commit 612c73e5bd will soon be integrated into staging-30. Use |
🚨 Branch Integration: This merge request has conflicts We couldn't automatically merge the commit 612c73e5bd into staging-30! You can use this resolution PR: #2893 to fix the conflicts. If you need support, contact us on Slack #devflow with those details! |
🚂 Branch Integration: starting soon, median merge time is 12m Commit 612c73e5bd will soon be integrated into staging-30. |
🚂 Branch Integration Commit 612c73e5bd has been merged into staging-30 in merge commit a6a057e8da. Check out the triggered pipeline on Gitlab 🦊 |
1ab31a8
to
612c73e
Compare
/to-staging |
🚂 Branch Integration: starting soon, median merge time is 11m Commit a6fc9ff046 will soon be integrated into staging-30. Use |
🚨 Branch Integration: This merge request has conflicts We couldn't automatically merge the commit a6fc9ff046 into staging-30! You can use this resolution PR: #2896 to fix the conflicts. If you need support, contact us on Slack #devflow with those details! |
🚂 Branch Integration: starting soon, median merge time is 11m Commit a6fc9ff046 will soon be integrated into staging-30. |
🚂 Branch Integration Commit a6fc9ff046 has been merged into staging-30 in merge commit ab08076032. Check out the triggered pipeline on Gitlab 🦊 |
@@ -226,6 +226,16 @@ | |||
- ♻️ [RUM-6180] Use performanceObserver for long-task entries ([#2989](https://github.com/DataDog/browser-sdk/pull/2989)) [RUM] [RUM-SLIM] | |||
- ⚗️✨ [RUM 5983] Add set view context apis ([#2967](https://github.com/DataDog/browser-sdk/pull/2967)) [LOGS] [RUM] [RUM-SLIM] [WORKER] | |||
|
|||
## v6.0.0-alpha.0 |
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.
🥜 nitpick: this is out-of-order but also missing lots of things, let's update it when doing the release
@@ -16,6 +16,13 @@ type ContainerElementToProcess = { | |||
path: string | |||
} | |||
|
|||
type sanitizedEvent = { |
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.
🥜 nitpick: We need a lint rule to make sure types are always UpperCamelCased
Caution
Before merging, do not forget to re-enable commented out ci jobs:
Motivation
Changes
Testing
I have gone over the contributing documentation.