-
-
Notifications
You must be signed in to change notification settings - Fork 755
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
Add module build #492
Add module build #492
Conversation
@edi9999 review please 😃 |
6d4643c
to
dca5e0b
Compare
@@ -41,5 +41,9 @@ node_modules | |||
|
|||
package-lock.json | |||
|
|||
# Browser Version | |||
browser/lib/*.js |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think the files should be deleted if they are in .gitignore (.gitignore only applies to additions, not to modifications)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
they were. it was actually just a whitespace issue. it was:
browser/lib/*.js
(space at the end, not ignored)
changed to:
browser/lib/*.js
(no space at the end, ignored)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good to merge?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The file browser/lib/jimp.min.js is still in the repository in your Pull Request :
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
oops! addressed this.
dca5e0b
to
00093b9
Compare
Added some docs too on the module build and using it with webpack too. 👍 |
Module build (still has import/export in output) for ESM and webpack. The future is now!
Also in this PR:
.babelrc.js
to determine babel setup far more easily without repeating oneselfbabel-preset-stage-0
and add plugins we actually use. thestage-
presets are being deprecated@babel/plugin-proposal-class-properties
@babel/plugin-syntax-object-rest-spread
karma-browserify
to make the test run ridiculously fast.browser-build.js
(I want to delete most of this since we don't use much of it, another PR)babel-plugin-source-map-support
for "development" env and not browserOld time running
npm run test
: 43.04sNew time running
npm run test
: 72s