Skip to content
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

Improve Bundling (switch to rollup) #1371

Open
2 tasks
leonardssh opened this issue Aug 25, 2021 · 0 comments · May be fixed by #1372
Open
2 tasks

Improve Bundling (switch to rollup) #1371

leonardssh opened this issue Aug 25, 2021 · 0 comments · May be fixed by #1372

Comments

@leonardssh
Copy link
Member

leonardssh commented Aug 25, 2021

I was sitting and looking over the NPM downloads of this library, and the size of the bundle jumped out at me.

image
I mean, 1.15 MB with 400 files for a library? Seems a little too much to me.

That's why I thought it would be cool to switch to rollup, like the other libraries.

Results: ~194kb and only 3 files, no more!
image

Also, to be able to merge all the .d.ts into single file, I used @microsoft/api-extractor, which imo is really good and efficient.

TODO:

  • Fix tests (or switch to JEST)
  • Improve CI
@leonardssh leonardssh linked a pull request Aug 25, 2021 that will close this issue
11 tasks
@leonardssh leonardssh linked a pull request Aug 25, 2021 that will close this issue
11 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant