build: Vendor Split Popular Packages #3499
Conversation
Bundle ReportChanges will increase total bundle size by 31.15kB (0.18%) ⬆️. This is within the configured threshold ✅ Detailed changes
|
Bundle ReportChanges will increase total bundle size by 31.15kB (0.18%) ⬆️. This is within the configured threshold ✅ Detailed changes
|
✅ Deploy preview for gazebo ready!Previews expire after 1 month automatically.
|
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 #3499 +/- ##
=======================================
Coverage 98.92% 98.92%
=======================================
Files 808 808
Lines 14411 14411
Branches 4101 4101
=======================================
Hits 14256 14256
Misses 148 148
Partials 7 7
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 #3499 +/- ##
=======================================
Coverage 98.92% 98.92%
=======================================
Files 808 808
Lines 14411 14411
Branches 4094 4101 +7
=======================================
Hits 14256 14256
Misses 148 148
Partials 7 7
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 #3499 +/- ##
=======================================
Coverage 98.92% 98.92%
=======================================
Files 808 808
Lines 14411 14411
Branches 4094 4101 +7
=======================================
Hits 14256 14256
Misses 148 148
Partials 7 7
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 #3499 +/- ##
=======================================
Coverage 99.09% 99.09%
=======================================
Files 806 806
Lines 14354 14354
Branches 4075 4075
=======================================
Hits 14224 14224
Misses 123 123
Partials 7 7
Continue to review full report in Codecov by Sentry.
|
| vendor_lodash: [ | ||
| 'lodash/get', | ||
| 'lodash/isEqual', | ||
| 'lodash/isNull', | ||
| 'lodash/isNil', | ||
| 'lodash/isEmpty', | ||
| 'lodash/isString', | ||
| 'lodash/isNumber', | ||
| 'lodash/isBoolean', | ||
| 'lodash/isUndefined', | ||
| 'lodash/isArray', | ||
| 'lodash/gt', | ||
| 'lodash/omit', | ||
| 'lodash/flatMap', | ||
| 'lodash/groupBy', | ||
| ], |
There was a problem hiding this comment.
would we be able to leverage the use of ['lodash'] from https://rollupjs.org/configuration-options/#output-manualchunks so that we don't need to list each individual function one by one?
There was a problem hiding this comment.
I attempted that, however it did not work and included everything. I think mainly because we're not using @rollup/plugin-node-resolve we're using whatever is built into Vite
There was a problem hiding this comment.
thanks for checking
There was a problem hiding this comment.
Wanted to double double check, tried using that plugin and it did not work, so we're stuck with this 😢
nicholas-codecov
deleted the
build-vendor-split-packages-that-are-used-site-wide
branch
Description
This PR utilizes a practice called "vendor splitting", where packages that are used site wide are split out into their own chunk files. This brings some optimizations because these files change rarely and are able to be cached after first load reducing the amount of JS being loaded on initial load when the user visits the site.
Notable Changes
rollupOptions.output.manualChunksto split out packages into their own chunks