Skip to content

Commit

Permalink
Suggest using babel to remove console.* calls
Browse files Browse the repository at this point in the history
Summary:
This PR adds a suggestion to the docs to use  the `transform-remove-console` babel plugin in production to remove `console.*` calls.
This information was previously in the docs, but was [removed](e759573) because the babel plugin [didn't work](#10412).
But now it's working well, as reported [here](#10412 (comment)), so it would be helpful to add the suggestion again.
Ideally, this would be done automatically, as I suggested in #8337
Closes #13651

Differential Revision: D4954872

Pulled By: hramos

fbshipit-source-id: 89ae1b813c50e678f0826f16ef88c8604e13d889
  • Loading branch information
Emilios1995 authored and facebook-github-bot committed Apr 26, 2017
1 parent f3dbddc commit 515f851
Showing 1 changed file with 11 additions and 0 deletions.
11 changes: 11 additions & 0 deletions docs/Performance.md
Expand Up @@ -73,6 +73,17 @@ This is unavoidable: a lot more work needs to be done at runtime to provide you
When running a bundled app, these statements can cause a big bottleneck in the JavaScript thread.
This includes calls from debugging libraries such as [redux-logger](https://github.com/evgenyrodionov/redux-logger),
so make sure to remove them before bundling.
You can also use this [babel plugin](https://babeljs.io/docs/plugins/transform-remove-console/) that removes all the `console.*` calls. You need to install it first with `npm i babel-plugin-transform-remove-console --save`, and then edit the `.babelrc` file under your project directory like this:
```json
{
"env": {
"production": {
"plugins": ["transform-remove-console"]
}
}
}
```
This will automatically remove all `console.*` calls in the release (production) versions of your project.

### `ListView` initial rendering is too slow or scroll performance is bad for large lists

Expand Down

0 comments on commit 515f851

Please sign in to comment.