From 0e7050ec86c1fb3ab5a3fb6c4500cfa42442ae0d Mon Sep 17 00:00:00 2001 From: Joyee Cheung Date: Sat, 20 Jan 2018 21:38:24 +0800 Subject: [PATCH] doc: improve the instructions of onboarding PR PR-URL: https://github.com/nodejs/node/pull/18268 Fixes: https://github.com/nodejs/node/issues/18090 Refs: https://github.com/nodejs/TSC/pull/472 Reviewed-By: Jon Moss Reviewed-By: Shingo Inoue Reviewed-By: Gireesh Punathil Reviewed-By: Ben Noordhuis Reviewed-By: Weijia Wang Reviewed-By: Minwoo Jung Reviewed-By: Luigi Pinca Reviewed-By: Matteo Collina Reviewed-By: Gibson Fahnestock Reviewed-By: Daijiro Wachi --- doc/onboarding.md | 12 +++++++----- 1 file changed, 7 insertions(+), 5 deletions(-) diff --git a/doc/onboarding.md b/doc/onboarding.md index e67f6638d3a0e4..42144e23822dd3 100644 --- a/doc/onboarding.md +++ b/doc/onboarding.md @@ -199,12 +199,13 @@ onboarding session. * Optionally, include your personal pronouns. * Label your pull request with the `doc` subsystem label. * Run CI on your PR. -* After one or two approvals, land the PR. +* After one or two approvals, land the PR (PRs of this type do not need to wait + for 48/72 hours to land). * Be sure to add the `PR-URL: ` and appropriate `Reviewed-By:` - metadata! - * [`core-validate-commit`][] helps a lot with this – install and use it if you - can! - * [`node-core-utils`][] fetches the metadata for you. + metadata. + * [`core-validate-commit`][] automates the validation of commit messages. + * [`node-core-utils`][] automates the generation of metadata and the landing + process. See the documentation of [`git-node`][]. ## Final notes @@ -227,6 +228,7 @@ onboarding session. [Code of Conduct]: https://github.com/nodejs/admin/blob/master/CODE_OF_CONDUCT.md [`core-validate-commit`]: https://github.com/evanlucas/core-validate-commit +[`git-node`]: https://github.com/nodejs/node-core-utils#git-node [`node-core-utils`]: https://github.com/nodejs/node-core-utils [Landing Pull Requests]: https://github.com/nodejs/node/blob/master/COLLABORATOR_GUIDE.md#landing-pull-requests [https://github.com/nodejs/node/commit/ce986de829457c39257cd205067602e765768fb0]: https://github.com/nodejs/node/commit/ce986de829457c39257cd205067602e765768fb0