-
Notifications
You must be signed in to change notification settings - Fork 9
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
[MOB-9520]: enable tree-shaking #451
Conversation
|
@mprew97 -- one question -- are we importing any pollyfills or css? (not including things like style-components) |
@mprew97 Unless I'm doing something wrong, this hasn't changed the bundle size :( |
@lposen , @ts-nguyen added a screenshot with bundle sizes pre/post tree shaking. My mistake, should have added that to begin with. I also added the analyzer to the wrong project. Tree shaking shouldn't change the bundle size of the Web SDK itself, but rather the apps it is used in (from my understanding). So need to test in |
JIRA Ticket(s) if any
Description
Enables tree shaking to resolve this issue
Test Steps
sideEffects: false
frompackage.json
.yarn webpack-analyze
fromreact-example
dirsideEffects:false
topackage.json
Before: 3.31 mb
After: 2.78 mb