diff options
| author | Gireesh Punathil <gpunathi@in.ibm.com> | 2020-07-21 05:44:45 +0300 |
|---|---|---|
| committer | Gireesh Punathil <gpunathi@in.ibm.com> | 2020-07-24 07:41:52 +0300 |
| commit | 983364c0ff3f4784a1aaab7c39a2abc54b3f9df0 (patch) | |
| tree | fb2f767f4928b01d77aaf6f412cc5e868c586292 /onboarding.md | |
| parent | adcf12c39f26627f0964e96863150467dc1e99ba (diff) | |
doc: onboarding process extras
Callout some practices explicitly, so that
the process is followed in a similar manner
PR-URL: https://github.com/nodejs/node/pull/34455
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Andrey Pechkurov <apechkurov@gmail.com>
Diffstat (limited to 'onboarding.md')
| -rw-r--r-- | onboarding.md | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/onboarding.md b/onboarding.md index 31b8353ef41..8ec268570e8 100644 --- a/onboarding.md +++ b/onboarding.md @@ -210,6 +210,9 @@ needs to be pointed out separately during the onboarding. * Run CI on the PR. Use the `node-test-pull-request` CI task. * After two Collaborator approvals for the change and two Collaborator approvals for fast-tracking, land the PR. +* Leave a comment in the PR: `Please 👍 this comment to approve fast-tracking`. +* If there are not enough approvals within a reasonable time, consider the + single approval of the onboarding TSC member sufficient, and land the PR. * Be sure to add the `PR-URL: <full-pr-url>` and appropriate `Reviewed-By:` metadata. * [`node-core-utils`][] automates the generation of metadata and the landing |