The core team will be monitoring for pull requests.
Before submitting a pull request, please make sure the following is done…
- Fork the repo and create your branch from
master
. - If you've added code that should be tested, add tests!
- If you've changed APIs, update the documentation.
- Make sure your code lints (
npm run lint
) - we've done our best to make sure these rules match our internal linting guidelines. - If you haven't already, complete the CLA.
In order to accept your pull request, we need you to submit a CLA. You only need to do this once, so if you've done this for another Facebook open source project, you're good to go. If you are submitting a pull request for the first time, just let us know that you have completed the CLA and we can cross-check with your GitHub username.
Complete your CLA here: https://developers.facebook.com/opensource/cla
We will be using GitHub Issues for our public bugs. We will keep a close eye on this and try to make it clear when we have an internal fix in progress. Before filing a new task, try to make sure your problem doesn't already exist.
The best way to get your bug fixed is to provide a reduced test case. jsFiddle, jsBin, and other sites provide a way to give live examples. Those are especially helpful though may not work for JSX
-based code.
Facebook has a bounty program for the safe disclosure of security bugs. With that in mind, please do not file public issues and go through the process outlined on that page.
- IRC - #reactjs on freenode
- Mailing list - reactjs on Google Groups
- Use semicolons;
- Commas last,
- 2 spaces for indentation (no tabs)
- Prefer
'
over"
"use strict";
- 80 character line length
- "Attractive"
Please npm run lint
.
By contributing to React, you agree that your contributions will be licensed under the attached License.