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
Share minified code only in production #59
Comments
ShareJS will serve the uncompressed version of the JS as well. While debugging, I just change my script reference to <script src="share/share.uncompressed.js"></script> Is NODE_ENV pretty standard now? You're right - that might be a better idea. |
Well at least Express uses it and it's pretty de facto :) But anyways consider this as documentation issue. I think by default we should point people to use the uncompressed one. Minified one comes to table only when the app is really in production. Before that we should make everything clear as possible for the developers. Maybe |
Yeah fair enough. For now lets make the documentation by default point to Mind if I assign this to you, @epeli ? |
Well, sure, go ahead and assign :) |
Closes issue josephg#59 (josephg#59) sed is my friend
Examples now use unminified version. Can this issue be closed? |
Oh, I totally forgot this. Yes, thank you! |
ShareJS should serve minified code only when NODE_ENV=production is set.
Or if there is a way to disable it, it should be documented better :)
I had today hard time debugging it.
The text was updated successfully, but these errors were encountered: