Permalink
Browse files

doc: update author-ready label terms

PR-URL: #23249
Refs: #22255
Reviewed-By: Refael Ackermann <refack@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Reviewed-By: Sakthipriyan Vairamani <thechargingvolcano@gmail.com>
  • Loading branch information...
vsemozhetbyt committed Oct 3, 2018
1 parent 061e098 commit 1a18e3572e5403f6f665232bcf70d7cbcc270994
Showing with 7 additions and 5 deletions.
  1. +5 −4 COLLABORATOR_GUIDE.md
  2. +2 −1 doc/onboarding.md
View
@@ -83,10 +83,11 @@ necessary.
### Author ready pull requests
A pull request that is still awaiting the minimum review time is considered
_author ready_ as soon as the CI has been started, it has at least one approval,
and it has no outstanding review comments. Please always make sure to add the
`author ready` label to the PR in that case and remove it again as soon as that
condition is not met anymore.
_author ready_ as soon as the CI has been started, it has at least two approvals
(one Collaborator approval is enough if the pull request has been open for more
than 7 days), and it has no outstanding review comments. Please always make sure
to add the `author ready` label to the PR in that case and remove it again as
soon as that condition is not met anymore.
### Handling own pull requests
View
@@ -99,7 +99,8 @@ onboarding session.
* Please add the `author-ready` label for PRs where:
* the CI has been started (not necessarily finished),
* no outstanding review comments exist and
* at least one collaborator approved the PR.
* at least two Collaborators approved the PR (one Collaborator approval is
enough if the pull request has been open for more than 7 days).
* See [Who to CC in the issue tracker][who-to-cc].
* This will come more naturally over time

0 comments on commit 1a18e35

Please sign in to comment.