forked from facebook/react-native
-
Notifications
You must be signed in to change notification settings - Fork 0
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Summary: Adds a new maintainers guide, and updates the contributor's guide to be consistent with regards to this new guide. Some additional style changes made in order to support the display of bot commands. Changed the wording for the "Edit this page on GitHub" link. Finally, the contributor's guide is now synced to `CONTRIBUTING.md` on the repo. ``` cd website && npm start ``` Verify that `CONTRIBUTING.md` is updated whenever the website is regenerated. Verify everything rendered correctly. Expand the details below to see screenshots. <details>   </details> Closes facebook#15202 Differential Revision: D5494246 Pulled By: hramos fbshipit-source-id: e28d5624d1e4795e212f10e8d5713d91a0eae15f
- Loading branch information
1 parent
f193c3f
commit aadeff0
Showing
13 changed files
with
589 additions
and
237 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,5 +1,220 @@ | ||
| # Contributing to React Native | ||
|
|
||
| Want to contribute to React Native? There are a few things you need to know. | ||
| <!-- generated_contributing_start --> | ||
| React Native is one of Facebook's first open source projects that is both under very active development and is also being used to ship code to everybody using Facebook's mobile apps. If you're interested in contributing to React Native, hopefully this document makes the process for contributing clear. | ||
|
|
||
| We wrote a [contribution guide](https://facebook.github.io/react-native/docs/contributing.html) to help you get started. | ||
| Core contributors to React Native meet monthly and post their meeting notes on the [React Native blog](https://facebook.github.io/react-native/blog). You can also find ad hoc discussions in the [React Native Core Contributors](https://www.facebook.com/groups/reactnativeoss/) Facebook group. | ||
|
|
||
| ## [Code of Conduct](https://code.facebook.com/codeofconduct) | ||
|
|
||
| Facebook has adopted a Code of Conduct that we expect project participants to adhere to. Please read [the full text](https://code.facebook.com/codeofconduct) so that you can understand what actions will and will not be tolerated. | ||
|
|
||
| ## How to contribute | ||
|
|
||
| There are many ways to contribute to React Native, and many of them do not involve writing any code. Here's a few ideas to get started: | ||
|
|
||
| * Simply start using React Native. Go through the [Getting Started](http://facebook.github.io/react-native/docs/getting-started.html) guide. Does everything work as expected? If not, we're always looking for improvements. Let us know by [opening an issue](http://facebook.github.io/react-native/docs/contributing.html#reporting-new-issues). | ||
| * Look through the [open issues](https://github.com/facebook/react-native/issues). Provide workarounds, ask for clarification, or suggest labels. Help [triage issues](http://facebook.github.io/react-native/docs/contributing.html#triaging-issues-and-pull-requests). | ||
| * If you find an issue you would like to fix, [open a pull request](http://facebook.github.io/react-native/docs/contributing.html#your-first-pull-request). Issues tagged as [`Good First Task`](https://github.com/facebook/react-native/labels/Good%20First%20Task) are a good place to get started. | ||
| * Read through the [React Native docs](http://facebook.github.io/react-native/docs). If you find anything that is confusing or can be improved, you can make edits by clicking "Improve this page" at the bottom of most docs. | ||
| * Browse [Stack Overflow](https://stackoverflow.com/questions/tagged/react-native) and answer questions. This will help you get familiarized with common pitfalls or misunderstandings, which can be useful when contributing updates to the documentation. | ||
| * Take a look at the [features requested](https://react-native.canny.io/feature-requests) by others in the community and consider opening a pull request if you see something you want to work on. | ||
|
|
||
| Contributions are very welcome. If you think you need help planning your contribution, please hop into [#react-native](https://discord.gg/0ZcbPKXt5bZjGY5n) and let people know you're looking for a mentor. | ||
|
|
||
| ### Triaging issues and pull requests | ||
|
|
||
| One great way you can contribute to the project without writing any code is to help triage issues and pull requests as they come in. | ||
|
|
||
| * Ask for more information if the issue does not provide all the details required by the template. | ||
| * Suggest [labels](https://github.com/facebook/react-native/labels) that can help categorize issues. | ||
| * Flag issues that are stale or that should be closed. | ||
| * Ask for test plans and review code. | ||
|
|
||
| Adding labels, closing and reopening issues, and merging pull requests is, as you may expect, limited to a subset of contributors. Simply commenting on the issue or pull request can still go a long way towards helping us keep the number of outstanding issues under control. | ||
|
|
||
| Once you have become an active contributor in the community, you may gain access to the Facebook GitHub Bot, allowing you to perform some of these operations yourself. You can learn more about the bot in the [maintainer's guide](docs/maintainers.html#facebook-github-bot). | ||
|
|
||
|
|
||
| ## Our development process | ||
|
|
||
| Some of the core team will be working directly on [GitHub](https://github.com/facebook/react-native). These changes will be public from the beginning. Other changesets will come via a bridge with Facebook's internal source control. This is a necessity as it allows engineers at Facebook outside of the core team to move fast and contribute from an environment they are comfortable in. | ||
|
|
||
| When a change made on GitHub is approved, it will first be imported into Facebook's internal source control. The change will eventually sync back to GitHub as a single commit once it has passed all internal tests. | ||
|
|
||
| ### Branch organization | ||
|
|
||
| We will do our best to keep `master` in good shape, with tests passing at all times. But in order to move fast, we will make API changes that your application might not be compatible with. We will do our best to [communicate these changes](https://github.com/facebook/react-native/releases) and version appropriately so you can lock into a specific version if need be. | ||
|
|
||
| To see what changes are coming and provide better feedback to React Native contributors, use the [latest release candidate](http://facebook.github.io/react-native/versions.html) when possible. By the time a release candidate is released, the changes it contains will have been shipped in production Facebook apps for over two weeks. | ||
|
|
||
| ## Bugs | ||
|
|
||
| We use [GitHub Issues](https://github.com/facebook/react-native/issues) for our public bugs. If you would like to report a problem, take a look around and see if someone already opened an issue about it. If you a are certain this is a new, unreported bug, you can submit a [bug report](http://facebook.github.io/react-native/docs/contributing.html#reporting-new-issues). | ||
|
|
||
| If you have questions about using React Native, the [help page](http://facebook.github.io/react-native/support.html) list various resources that should help you get started. | ||
|
|
||
| We also have a [place where you can request features or enhancements](https://react-native.canny.io/feature-requests). If you see anything you'd like to be implemented, vote it up and explain your use case. | ||
|
|
||
| ## Reporting new issues | ||
|
|
||
| When [opening a new issue](https://github.com/facebook/react-native/issues/new), always make sure to fill out the [issue template](https://raw.githubusercontent.com/facebook/react-native/master/.github/ISSUE_TEMPLATE.md). **This step is very important!** Not doing so may result in your issue getting closed. Don't take this personally if this happens, and feel free to open a new issue once you've gathered all the information required by the template. | ||
|
|
||
| * **One issue, one bug:** Please report a single bug per issue. | ||
| * **Provide a Snack:** The best way to get attention on your issue is to provide a reduced test case. You can use [Snack](https://snack.expo.io/) to demonstrate the issue. | ||
| * **Provide reproduction steps:** List all the steps necessary to reproduce the issue. Provide a Snack or upload a sample project to GitHub. The person reading your bug report should be able to follow these steps to reproduce your issue with minimal effort. | ||
| * **Try out the latest version:** Verify that the issue can be reproduced locally by updating your project to use [React Native from `master`](http://facebook.github.io/react-native/versions.html). The bug may have already been fixed! | ||
|
|
||
| We're not able to provide support through GitHub Issues. If you're looking for help with your code, consider asking on [Stack Overflow](http://stackoverflow.com/questions/tagged/react-native) or reaching out to the community through [other channels](https://facebook.github.io/react-native/support.html). | ||
|
|
||
| ### Security bugs | ||
|
|
||
| Facebook has a [bounty program](https://www.facebook.com/whitehat/) for the safe disclosure of security bugs. With that in mind, please do not file public issues; go through the process outlined on that page. | ||
|
|
||
| ## Pull requests | ||
|
|
||
| ### Your first pull request | ||
|
|
||
| So you have decided to contribute code back to upstream by opening a pull request. You've invested a good chunk of time, and we appreciate it. We will do our best to work with you and get the PR looked at. | ||
|
|
||
| Working on your first Pull Request? You can learn how from this free video series: | ||
|
|
||
| [**How to Contribute to an Open Source Project on GitHub**](https://egghead.io/courses/how-to-contribute-to-an-open-source-project-on-github) | ||
|
|
||
| We have a list of [beginner friendly issues](https://github.com/facebook/react-native/labels/Good%20First%20Task) to help you get your feet wet in the React Native codebase and familiar with our contribution process. This is a great place to get started. | ||
|
|
||
| ### Proposing a change | ||
|
|
||
| If you would like to request a new feature or enhancement but are not yet thinking about opening a pull request, we have a [place to track feature requests](https://react-native.canny.io/feature-requests). | ||
|
|
||
| If you intend to change the public API, or make any non-trivial changes to the implementation, we recommend [filing an issue](https://github.com/facebook/react-native/issues/new). This lets us reach an agreement on your proposal before you put significant effort into it. | ||
|
|
||
| If you're only fixing a bug, it's fine to submit a pull request right away but we still recommend to file an issue detailing what you're fixing. This is helpful in case we don't accept that specific fix but want to keep track of the issue. | ||
|
|
||
| ### Sending a pull request | ||
|
|
||
| Small pull requests are much easier to review and more likely to get merged. Make sure the PR does only one thing, otherwise please split it. | ||
|
|
||
| **Before submitting a pull request**, please make sure the following is done: | ||
|
|
||
| 1. Fork [the repository](https://github.com/facebook/react-native) and create your branch from `master`. | ||
| 2. Add the copyright notice to the top of any new files you've added. | ||
| 3. Describe your [**test plan**](https://facebook.github.io/react-native/docs/contributing.html#test-plan) in your commit. | ||
| 4. Ensure [**tests pass**](https://facebook.github.io/react-native/docs/contributing.html#contrinuous-integration-tests) on both Travis and Circle CI. | ||
| 5. Make sure your code lints (`npm run lint`). | ||
| 6. If you haven't already, [sign the CLA](https://code.facebook.com/cla). | ||
|
|
||
| All pull requests should be opened against the `master` branch. | ||
|
|
||
| > **Note:** It is not necessary to keep clicking `Merge master to your branch` on the PR page. You would want to merge master if there are conflicts or tests are failing. The Facebook-GitHub-Bot ultimately squashes all commits to a single one before merging your PR. | ||
| #### Test plan | ||
|
|
||
| A good test plan has the exact commands you ran and their output, provides screenshots or videos if the pull request changes UI or updates the website. | ||
|
|
||
| * If you've added code that should be tested, add tests! | ||
| * If you've changed APIs, update the documentation. | ||
| * If you've updated the docs, verify the website locally and submit screenshots if applicable (see [website/README.md](https://github.com/facebook/react-native/blob/master/website/README.md)) | ||
|
|
||
| See [What is a Test Plan?](https://medium.com/@martinkonicek/what-is-a-test-plan-8bfc840ec171#.y9lcuqqi9) to learn more. | ||
|
|
||
| #### Continuous integration tests | ||
|
|
||
| Make sure all **tests pass** on both [Travis][travis] and [Circle CI][circle]. PRs that break tests are unlikely to be merged. Learn more about [testing your changes here](https://facebook.github.io/react-native/docs/testing.html). | ||
|
|
||
| [travis]: https://travis-ci.org/facebook/react-native | ||
| [circle]: http://circleci.com/gh/facebook/react-native | ||
|
|
||
| #### Breaking changes | ||
|
|
||
| When adding a new breaking change, follow this template in your pull request: | ||
|
|
||
| ``` | ||
| ### New breaking change here | ||
| * **Who does this affect**: | ||
| * **How to migrate**: | ||
| * **Why make this breaking change**: | ||
| * **Severity (number of people affected x effort)**: | ||
| ``` | ||
|
|
||
| If your pull request is merged, a core contributor will update the [list of breaking changes](https://github.com/facebook/react-native/wiki/Breaking-Changes) which is then used to populate the release notes. | ||
|
|
||
| #### Copyright Notice for files | ||
|
|
||
| Copy and paste this to the top of your new file(s): | ||
|
|
||
| ```JS | ||
| /** | ||
| * Copyright (c) 2015-present, Facebook, Inc. | ||
| * All rights reserved. | ||
| * | ||
| * This source code is licensed under the BSD-style license found in the | ||
| * LICENSE file in the root directory of this source tree. An additional grant | ||
| * of patent rights can be found in the PATENTS file in the same directory. | ||
| */ | ||
| ``` | ||
|
|
||
| If you've added a new module, add a `@providesModule <moduleName>` at the end of the comment. This will allow the haste package manager to find it. | ||
|
|
||
| #### Contributor License Agreement (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, the Facebook GitHub Bot will reply with a link to the CLA form. You may also [complete your CLA here](https://code.facebook.com/cla). | ||
|
|
||
| ### What happens next? | ||
|
|
||
| The core team will be monitoring for pull requests. Read [what to expect from maintainers](https://facebook.github.io/react-native/docs/maintainers.html#handling-pull-requests) to understand what may happen after you open a pull request. | ||
|
|
||
| ## Style Guide | ||
|
|
||
| Our linter will catch most styling issues that may exist in your code. You can check the status of your code styling by simply running `npm run lint`. | ||
|
|
||
| However, there are still some styles that the linter cannot pick up. | ||
|
|
||
| ### Code Conventions | ||
|
|
||
| #### General | ||
|
|
||
| * **Most important: Look around.** Match the style you see used in the rest of the project. This includes formatting, naming things in code, naming things in documentation. | ||
| * Add trailing commas, | ||
| * 2 spaces for indentation (no tabs) | ||
| * "Attractive" | ||
|
|
||
| #### JavaScript | ||
|
|
||
| * Use semicolons; | ||
| * `'use strict';` | ||
| * Prefer `'` over `"` | ||
| * Do not use the optional parameters of `setTimeout` and `setInterval` | ||
| * 80 character line length | ||
|
|
||
| #### JSX | ||
|
|
||
| * Prefer `"` over `'` for string literal props | ||
| * When wrapping opening tags over multiple lines, place one prop per line | ||
| * `{}` of props should hug their values (no spaces) | ||
| * Place the closing `>` of opening tags on the same line as the last prop | ||
| * Place the closing `/>` of self-closing tags on their own line and left-align them with the opening `<` | ||
|
|
||
| #### Objective-C | ||
|
|
||
| * Space after `@property` declarations | ||
| * Brackets on *every* `if`, on the *same* line | ||
| * `- method`, `@interface`, and `@implementation` brackets on the following line | ||
| * *Try* to keep it around 80 characters line length (sometimes it's just not possible...) | ||
| * `*` operator goes with the variable name (e.g. `NSObject *variableName;`) | ||
|
|
||
| #### Java | ||
|
|
||
| * If a method call spans multiple lines closing bracket is on the same line as the last argument. | ||
| * If a method header doesn't fit on one line each argument goes on a separate line. | ||
| * 100 character line length | ||
|
|
||
| ### Documentation | ||
|
|
||
| * Do not wrap lines at 80 characters - configure your editor to soft-wrap when editing documentation. | ||
|
|
||
| ## License | ||
|
|
||
| By contributing to React Native, you agree that your contributions will be licensed under its BSD license. | ||
| <!-- generated_contributing_end --> |
Oops, something went wrong.