-
Notifications
You must be signed in to change notification settings - Fork 103
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
NPM package #57
Comments
You can already install it over npm by specifying the git repo as there's a package.json and index. I would generally be fine with the npm registry listing this repo but just be aware that you run a security risk by having that layer of automation... something I would tend to try to avoid recommending for such a sensitive library. |
Yes, you are right about installing git repo using NPM, however, the master branch is not ES5 friendly (needs a bable build with a dist folder) and I'm not sure if cpp branch is production ready? |
The master branch has been built to es5 successfully by many people… Cpp is close to or at production readiness. More people should test it. |
What would be done to the cpp branch which could not be done to the master branch anyway? I would accept a working PR . |
Yes I also put it in my webpack build but it's adding extra time to my environment build time and many standard webpack builds would ignore |
Feel free to submit fixes via PR. This is a community project :) |
Sure, will submit soon |
Is it possible to have an NPM package published for this repository? Preferably with a babel build script to publish the ES5 code so it wouldn't be rejected by React build environments.
The text was updated successfully, but these errors were encountered: