-
Notifications
You must be signed in to change notification settings - Fork 23
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
feat: Add in bundle chart #2984
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
✅ All tests successful. No failed tests found. @@ Coverage Diff @@
## main #2984 +/- ##
=======================================
Coverage 98.36% 98.36%
=======================================
Files 890 892 +2
Lines 13257 13267 +10
Branches 3468 3534 +66
=======================================
+ Hits 13040 13050 +10
Misses 213 213
Partials 4 4
Continue to review full report in Codecov by Sentry.
|
Codecov ReportAll modified and coverable lines are covered by tests ✅ ✅ All tests successful. No failed tests found. @@ Coverage Diff @@
## main #2984 +/- ##
=======================================
Coverage 98.36% 98.36%
=======================================
Files 890 892 +2
Lines 13257 13267 +10
Branches 3538 3469 -69
=======================================
+ Hits 13040 13050 +10
Misses 213 213
Partials 4 4
Continue to review full report in Codecov by Sentry.
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
✅ All tests successful. No failed tests found @@ Coverage Diff @@
## main #2984 +/- ##
=======================================
Coverage 98.36% 98.36%
=======================================
Files 890 892 +2
Lines 13257 13267 +10
Branches 3514 3469 -45
=======================================
+ Hits 13040 13050 +10
Misses 213 213
Partials 4 4
Continue to review full report in Codecov by Sentry.
|
Bundle ReportChanges will increase total bundle size by 16.45kB ⬆️
|
Bundle ReportChanges will increase total bundle size by 16.45kB ⬆️
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
✅ All tests successful. No failed tests found. Additional details and impacted files@@ Coverage Diff @@
## main #2984 +/- ##
===========================================
Coverage 98.36000 98.36000
===========================================
Files 890 892 +2
Lines 13257 13267 +10
Branches 3533 3515 -18
===========================================
+ Hits 13040 13050 +10
Misses 213 213
Partials 4 4
Continue to review full report in Codecov by Sentry.
|
6f18d95
to
06eafce
Compare
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.
lgtm
Description
This PR puts all of the pieces together and adds in the bundle chart component to the bundles tab.
Blocked by #2983
Closes codecov/engineering-team#1802
Notable Changes
TrendDropdown
component for bundle selectingBundleChart
componentBundleChart
toBundleContent
BundleSummary
stylesScreenshots