Skip to content
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

Merged
merged 3 commits into from
Mar 3, 2017
Merged

Added Brunch build tool to the docs #9074

merged 3 commits into from
Mar 3, 2017

Conversation

beznosd
Copy link
Contributor

@beznosd beznosd commented Feb 27, 2017

#### 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.
Copy link
Collaborator

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?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Definitely yes

Copy link
Contributor

@aweary aweary left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just some slight grammar tweaks, LGTM otherwise!

#### 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.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's rephrase this to:

To create an optimized product build with Brunch, just add the -p flag to the build command. See the Brunch docs for more details.

@@ -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.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lets rephrase to:

For Brunch, you need to add the -p flag to the build command.

@aweary
Copy link
Contributor

aweary commented Mar 3, 2017

Thanks @beznosd, looks great! I realized I had a typo in my recommended changes so I went ahead and fixed it for you. Sorry about that 😄

@aweary aweary merged commit 0bedd06 into facebook:master Mar 3, 2017
@beznosd
Copy link
Contributor Author

beznosd commented Mar 3, 2017

Cool! It's okay. Thanks @aweary!

@gaearon
Copy link
Collaborator

gaearon commented Mar 4, 2017

@aweary Can you send a PR that reorders them by alphabet, except for CRA which should go first? Otherwise it's a bit odd that Brunch goes first considering it's much less popular in the React community.

gaearon pushed a commit that referenced this pull request Mar 9, 2017
* Added Brunch build tool to the docs

* Improved grammar

* product build -> production build

(cherry picked from commit 0bedd06)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants