From 2f836e76bdac2cf96b4dad5b8bb04616b3618fe5 Mon Sep 17 00:00:00 2001 From: Rich Trott Date: Sat, 17 Feb 2018 19:25:07 -0800 Subject: [PATCH] doc: update onboarding.md for faster exercise completion MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Note that the CI run for the exercise can be minimal. PR-URL: https://github.com/nodejs/node/pull/18846 Reviewed-By: Richard Lau Reviewed-By: Luigi Pinca Reviewed-By: Ruben Bridgewater Reviewed-By: Michaƫl Zasso Reviewed-By: Matheus Marchini Reviewed-By: Jon Moss Reviewed-By: Gibson Fahnestock Reviewed-By: Joyee Cheung Reviewed-By: Colin Ihrig Reviewed-By: Anna Henningsen Reviewed-By: Michael Dawson --- doc/onboarding.md | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/doc/onboarding.md b/doc/onboarding.md index 296f9db7d02ef8..e2a05f2ccd17eb 100644 --- a/doc/onboarding.md +++ b/doc/onboarding.md @@ -200,7 +200,10 @@ onboarding session. * Collaborators are in alphabetical order by GitHub username. * Optionally, include your personal pronouns. * Label your pull request with the `doc` subsystem label. -* Run CI on your PR. +* Run CI on the PR. Because the PR does not affect any code, use the + `node-test-pull-request-lite` CI task. Alternatively, use the usual + `node-test-pull-request` CI task and cancel it after the linter and one other + subtask have passed. * 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:`