-
Notifications
You must be signed in to change notification settings - Fork 28
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
Use of console.log()
#7
Comments
@maxkfranz great tip. thanks. |
@maxkfranz I implemented a simple version of your suggestion. Adequate? It avoids adding another npm module. Manipulation of development|production status is hard-coded, but perhaps not too awful.
|
If you're fine with hard-coding that value, that's fine. Otherwise it's best to use something like
|
@maxkfranz I'll give it a try. webpack plugins are a little opaque to me at this point. Are these steps about right?
Am I close? |
It should be built-in
yes
yes
yes
|
While useful for debugging, these should be removed at least in prod builds.
You can use NODE_ENV and the environment webpack plugin for this, e.g.
An approach like above should be used or the messages should be removed altogether.
The text was updated successfully, but these errors were encountered: