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

Large number of dependecies #569

Closed
zaiste opened this issue Mar 6, 2020 · 2 comments
Closed

Large number of dependecies #569

zaiste opened this issue Mar 6, 2020 · 2 comments
Labels

Comments

@zaiste
Copy link
Contributor

zaiste commented Mar 6, 2020

I've added pure-engine to one of my projects and the node_modules size increased by 25%. There are several dependencies directly in pure-engine which seem unnecessary e.g. rollup et al.

Would it be possible to reduce that?

@emilos
Copy link
Contributor

emilos commented Mar 6, 2020

It's something that should definitely be looked into, I haven't spent a lot of time trying to narrow it down.

rollup or any other bundler might be necessary for scoped js, which import deps, the compiler has some complexities that are hard to avoid

@emilos emilos added the chore label Mar 6, 2020
@emilos
Copy link
Contributor

emilos commented Mar 16, 2020

Some small ideas for improvements:

  • add asttv as a AbstractSyntaxTree.serialize method (https://github.com/buxlabs/abstract-syntax-tree)
  • remove words-to-numbers dep and related code/specs
  • check if babel/core is still required as a dev dep, remove if it's unnecessary
  • try to update other deps
  • remove marked and related code (that was a bad choice, not using it right now anyway)
  • rewrite/include escape-html dep

@emilos emilos closed this as completed Sep 7, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants