-
-
Notifications
You must be signed in to change notification settings - Fork 161
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
build: disable minification #569
base: master
Are you sure you want to change the base?
Conversation
Why is minification not benficial in Node? |
Thank you for considering my suggestion! While minified code might still have some minimal runtime advantages due to optimization heuristics, (even though the |
It does. The package needs to be downloaded. And since it gets stored to disk long-term, size probably matters more. |
My 2 cents: the ability to drop into |
FWIW: #391 |
There seems to be little need for minification, as this is a
node
based project never intended to be run in browsers. My reason to get rid of it would be to enable the usage of tools likepatch-package
on the library, which operate on the distributed source, and is practically incompatible withtsx
in its current state.