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

Don't uglify NPM package source files #6

Closed
kyledrake opened this issue Mar 9, 2014 · 7 comments
Closed

Don't uglify NPM package source files #6

kyledrake opened this issue Mar 9, 2014 · 7 comments

Comments

@kyledrake
Copy link

The NPM source is uglified, which makes it difficult to debug errors when using it.

I think it's best to not uglify the package, and people interested in doing so for browser usage can then run it through browserify and uglify when needed.

This is related to bitcoinjs/bitcoinjs-lib#57

@weilu
Copy link

weilu commented Mar 9, 2014

👍

ghost pushed a commit that referenced this issue Mar 21, 2014
@ghost
Copy link

ghost commented Mar 22, 2014

Unfortunately, I can't publish the changed package.

on npm login I get the following error:

npm http PUT https://registry.npmjs.org/-/user/org.couchdb.user:evanvosberg
npm http 503 https://registry.npmjs.org/-/user/org.couchdb.user:evanvosberg
npm ERR! registry error parsing json

Login on the npm web-page works but not in the terminal ... ((

@kyledrake
Copy link
Author

Thank you for doing this! You are awesome.

There appear to be a downtime at the moment, but they should resolve it soon: https://twitter.com/npmjs/status/423185634581176320

Please let us know when you can push and we will quickly get this implemented. Thanks!

@kyledrake
Copy link
Author

Err, hmm, that's an old tweet. The status seems to be working: http://status.npmjs.org/

Maybe try upgrading npm to the latest version?

@ghost
Copy link

ghost commented Mar 22, 2014

Published!

@ghost ghost closed this as completed Mar 22, 2014
weilu added a commit to bitcoinjs/bitcoinjs-lib that referenced this issue Mar 22, 2014
@weilu
Copy link

weilu commented Mar 22, 2014

@kyledrake
Copy link
Author

Thank you! 🤘

@ghost ghost added the enhancement label Jun 25, 2014
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants