Skip to content

Commit

Permalink
Add docs on when reviews should be cleared (#583)
Browse files Browse the repository at this point in the history
  • Loading branch information
MrAlias committed Mar 5, 2021
1 parent 542d974 commit 3fcec94
Showing 1 changed file with 7 additions and 1 deletion.
8 changes: 7 additions & 1 deletion CONTRIBUTING.md
Expand Up @@ -75,7 +75,13 @@ A PR is considered to be **ready to merge** when:

* It has received two approvals from Approvers/Maintainers (at
different companies).
* Major feedback items are resolved.
* Feedback has been addressed.
* Any substantive changes to your PR will require that you clear any prior
Approval reviews, this includes changes resulting from other feedback. Unless
the approver explicitly stated that their approval will persist across
changes it should be assumed that the PR needs their review again. Other
project members (e.g. approvers, maintainers) can help with this if there are
any questions or if you forget to clear reviews.
* It has been open for review for at least one working day. This gives
people reasonable time to review.
* Trivial change (typo, cosmetic, doc, etc.) doesn't have to wait for
Expand Down

0 comments on commit 3fcec94

Please sign in to comment.