Skip to content

Commit 788e395

Browse files
committed
workflow: reviewers dismissed update
1 parent 52c8859 commit 788e395

File tree

1 file changed

+2
-0
lines changed

1 file changed

+2
-0
lines changed

docs/reference/contributing/guidelines/workflow.md

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -161,6 +161,8 @@ Each state is time boxed. In most cases, this is sufficient time to move to anot
161161

162162
All pull requests must be reviewed. The Arm Mbed CI bot determines the most suitable person to review the pull request and tags that person accordingly.
163163

164+
Github dismissed reviewers status after any change to the pull request commit history (adding new commit, rebase). Simple changes like documentation edits or rebase on top of latest master should only require rereview by maintainers. Their approval is sufficient considering it was already approved by a team assigned as a reviewer.
165+
164166
Time: 3 days for reviewers to leave feedback after the maintainers add the "needs: review" label.
165167

166168
#### The CI (Continuous Integration) testing

0 commit comments

Comments
 (0)