-
Notifications
You must be signed in to change notification settings - Fork 47.3k
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
Added Brunch build tool to the docs #9074
Changes from 1 commit
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -99,6 +99,11 @@ By default, React includes many helpful warnings. These warnings are very useful | |
|
||
If you use [Create React App](https://github.com/facebookincubator/create-react-app), `npm run build` will create an optimized build of your app in the `build` folder. | ||
|
||
#### Brunch | ||
|
||
Just add flag `-p` to the command to create an optimized production build. | ||
For example: `brunch build -p`. See more details in [Brunch docs](http://brunch.io/docs/commands) commands section. | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Let's rephrase this to:
|
||
|
||
#### Webpack | ||
|
||
Include both `DefinePlugin` and `UglifyJsPlugin` into your production Webpack configuration as described in [this guide](https://webpack.js.org/guides/production-build/). | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -13,6 +13,7 @@ If you're benchmarking or experiencing performance problems in your React apps, | |
|
||
* For Create React App, you need to run `npm run build` and follow the instructions. | ||
* For single-file builds, we offer production-ready `.min.js` versions. | ||
* For Brunch, you need to add flag `-p` to the command. | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Lets rephrase to:
|
||
* For Browserify, you need to run it with `NODE_ENV=production`. | ||
* For Webpack, you need to add this to plugins in your production config: | ||
|
||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Does this ensure
process.env.NODE_ENV
get envified?There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Definitely yes