-
Notifications
You must be signed in to change notification settings - Fork 27
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
Migrating from browserify to rollup #89
Conversation
Hi @sibiraj-s, can you please adjust the PR title and description so it gets easier to understand that this is about "change the build setup to rollup for faster / easier builds"? That would be great =) |
Sorry. done. |
No problem, thank you very much =) |
cc @XhmikosR |
Note - adding |
Can I go ahead and merge this? |
I took a quick look and the changes look good to me. Sure, you can go ahead. |
Thanks. @DanielRuf |
Migrates from browserify to rollup.
New Files created.
dist/htmlminifier.umd.bundle.js
(for browsers)dist/htmlminifier.umd.bundle.min.js
(for browsers minified)dist/htmlminifier.esm.bundle.js
(for scripts using modules in browsers)dist/htmlminifier.js
(for nodejs using import script)Also supports new exports syntax. we should be able to provide conditional export of file and also later switch to module without breaking changes.
Breaking change:
dist/htmlminifier.js
is now ESM, the browser version can be found inumd/esm.bundle.js
.require
to the window scope.Note: the dist files are not published to npm. hence its very hightly unlike to affect end users.