Pull Requests & Reviews¶
How to Open a PR¶
You can create a Pull Request (PR) by following these steps. Remember that:
- The target branch should be
master
. - If your PR closes and issue, write "Closes
#<issue-id>
".
Your PR will be reviewed soon (refer to this page to learn more about reviews).
Before opening a PR please self-review your changes in GitHub and ensure that you follow our style guide to speed up the review process⚡
How to Incorporate the Reviewer's Feedback to your PR¶
It might be directly approved and merged or one of our reviewers will send you some comments and suggested changes.
When reviewers suggest changes in a pull request, you can automatically incorporate the changes into your PR.
- Apply any Suggested Changes using the "Commit suggestions" button.
- In order to process a comment, simply make modifications directly to your branch and push normally. GitHub will automatically detect these changes and add them to your existing PR.
- As you update your PR and apply changes, mark each conversation as resolved.
- If you run into any merge issues, refer to this page to learn how to resolve them.
NOTE: Remember to regularly sync your fork with the upstream repo:
git pull upstream/master
How to Review a PR¶
If you'd like to review an open PR please follow these steps:
Step 1: Comment and Suggest Changes¶
You can enter single or multi-line comments (click and drag to select the range of lines):
Always prefer making "Suggested Changes" using the ±
button:
If the suggestion you'd like to make cannot be expressed using "suggested changes" please enter a clear comment explaining what should be fixed (e.g. some paragraphs don't link properly or some essential information cannot be found and should be added).
Using "Suggested Changes" saves you as a reviewer and the PR author a lot of time. And you get points (attributions) for the changes that you suggested (if the author commits them you become a co-author of those commits). If you're constant with your reviewer work you can apply to be recognize as an official reviewer in our Acknowledgements page.
Step 2: Submit your Review¶
Once you went through the whole PR you can submit your review
- Click on "Review changes".
- Enter a comment for the contributor.
- Select the type of review you'd like to leave (Comment, Approve or Request Changes).
- Click on "Submit review".
Learn more: "(GitHub Docs) Reviewing proposed changes in a pull request".