Skip to content

(build) Externalize apollo and graphql libraries #722

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

Merged
merged 1 commit into from
Sep 3, 2019
Merged

(build) Externalize apollo and graphql libraries #722

merged 1 commit into from
Sep 3, 2019

Conversation

Austio
Copy link
Contributor

@Austio Austio commented Aug 7, 2019

This PR externalizes the peer dependencies in vue-apollo in order to reduce the bundle size of vue-apollo in the default umd build. Without this, all of those libraries are being included and compiled in a way that is not tree-shakable, which causes the bundle size to be quite large.

Closes #660

Library Before After WithExternals
esm 133k 57k
min 52k 26k
umd 142k 61k
# Current Build
drwxrwxr-x  2 talos talos   4096 Jul  5 11:50 .
drwxrwxr-x 16 talos talos   4096 Aug  7 10:26 ..
-rw-rw-r--  1 talos talos 133190 Aug  7 10:27 vue-apollo.esm.js
-rw-rw-r--  1 talos talos  52935 Aug  7 10:26 vue-apollo.min.js
-rw-rw-r--  1 talos talos 142186 Aug  7 10:27 vue-apollo.umd.js

# With externals
drwxrwxr-x  2 talos talos  4096 Jul  5 11:50 .
drwxrwxr-x 16 talos talos  4096 Aug  7 10:26 ..
-rw-rw-r--  1 talos talos 57047 Aug  7 10:28 vue-apollo.esm.js
-rw-rw-r--  1 talos talos 26922 Aug  7 10:28 vue-apollo.min.js
-rw-rw-r--  1 talos talos 61127 Aug  7 10:28 vue-apollo.umd.js

@Austio
Copy link
Contributor Author

Austio commented Aug 26, 2019

@Akryum Anything else you are needing to merge this feller in?

@Akryum Akryum merged commit ffeec48 into vuejs:master Sep 3, 2019
@Akryum
Copy link
Member

Akryum commented Sep 3, 2019

Thanks! 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Reduce Bundle Size
2 participants