Anyone is free to review and comment on any open pull requests.
All pull requests must be reviewed and approved by at least one core member before merge.
Very small pull requests may be merged with just one review if they:
Larger pull requests must meet at least one of the following two additional requirements.
New pull requests should be assigned to a core member for review within 3 business days of being opened. That person should either approve the changes or request changes within 4 business days of being assigned. If they’re too busy, they should assign it to someone else who can review it within that timeframe.
If the changes are small, that member can merge the PR once the changes are complete. Otherwise, they should assign the pull request to another member for a second review.
The pull request can then be merged whenever the second reviewer approves, or if 5 business days pass with no farther changes requested by anybody, whichever comes first.
Code reviews should focus on things which cannot be validated by machines.
Some examples include: