Pull request guidelines
This guide is intended for the web.dev teammates who review and merge content pull requests (PRs). These guidelines only apply to PRs that add or update content. They do not apply to engineering PRs. The purpose of this guide is to avoid duplication of reviewer effort and to ensure that each reviewer is properly labeling each PR for our automated reporting systems.
Claim a PR #
Use GitHub's Assignees field to indicate to the other PR reviewers that you are reviewing the PR.
Claim an inactive PR #
If a reviewer claims a PR but is inactive on the PR for 3 business days, any other reviewer can reclaim the PR. The new reviewer should unassign the old reviewer and reassign themselves.
Properly label the PR #
New content #
Make sure that there's a comment in the PR that uses GitHub's issue closing keyword. In other words, there should be a comment along the lines of Fixes #45
, where #45
is the tracking issue for the new content.
If the PR for the new content doesn't have a tracking issue, contact the web.dev content lead.
Content updates #
Add the content update
label to the PR.
Add one of the following labels to the PR:
performance
frontend
privacy/security
capabilities
pwa
Contact the web.dev content lead if you don't know what label to use.
Presubmit #
Pull requests must have the presubmit workflow run in order to ensure that they do not break the site. The workflow only runs when the $-presubmit
label is added to the PR.
When a PR us ready to be merged in, add the $-presubmit
label and wait to see if the workflow passes.
Don't merge the PR until you're 100% certain that it's OK to merge #
Once a PR is merged, the new or updated content is deployed to the site within minutes. When in doubt, get an explicit confirmation of the publication date from the author.