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](facebook@e759573) because the babel plugin [didn't work](facebook#10412).
But now it's working well, as reported [here](facebook#10412 (comment)), so it would be helpful to add the suggestion again.
Ideally, this would be done automatically, as I suggested in facebook#8337
Closes facebook#13651

Differential Revision: D4954872

Pulled By: hramos

fbshipit-source-id: 89ae1b813c50e678f0826f16ef88c8604e13d889
  • Loading branch information
Emilios1995 authored and thotegowda committed May 7, 2017
1 parent f7cab4e commit be39f0c
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 be39f0c

Please sign in to comment.