-
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
⚡️Reduce bundle size: simplify config, use const enum, remove str redundancies #2877
Conversation
c247083
to
bf005bd
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #2877 +/- ##
==========================================
+ Coverage 93.68% 93.75% +0.07%
==========================================
Files 266 266
Lines 7584 7575 -9
Branches 1687 1685 -2
==========================================
- Hits 7105 7102 -3
+ Misses 479 473 -6 ☔ View full report in Codecov by Sentry. |
Bundles Sizes Evolution
🚀 CPU Performance
🧠 Memory Performance
|
c316478
to
5a46aa6
Compare
5a46aa6
to
7d552bd
Compare
/to-staging |
🚂 Branch Integration: starting soon, median merge time is 0s Commit 49a8bac5c1 will soon be integrated into staging-30. Use |
Integrated commit sha: 49a8bac Co-authored-by: Aymeric Mortemousque <aymeric.mortemousque@datadoghq.com>
🚂 Branch Integration: This commit was successfully integrated Commit 49a8bac5c1 has been merged into staging-30 in merge commit 2991a110a8. Check out the triggered pipeline on Gitlab 🦊 |
Motivation
Reduce bundle size
Changes
Testing
I have gone over the contributing documentation.