Guidelines for Reviewing and Merging PRs

Reviewing a PR

• If it's a simple typo, URL, or grammatical fix, then:
  1. Make sure it's correct.
  2. Check the preview to verify that the page renders correctly and/or the links work.
  3. Add the label Docs LGTM.
• For something more lengthy:
  1. Do an edit pass.
  2. Suggest any changes in a review.
  3. If there are any outstanding issues that need to be resolved, add the Docs Review: Open Issues label.
• Anything that you are not sure about the technical correctness of:
  1. Add the Needs Tech Review label.
  2. Assign a reviewer based on who is listed as a reviewer in the file or as appropriate from the Tech reviewers list.

Once a PR has a Docs LGTM and, if needed, a Tech LGTM, you can then merge it or write a comment to @chenopis or @zacharysarah to tell one of them to merge it.

Merging a PR

• If there are multiple commits, use "Squash and merge."
• Otherwise, use "Rebase and merge."
• Only use "Merge pull request" if the branch is up-to-date and there is only one commit.