Commit 58ecdd4
doc: update onboarding.md for faster exercise completion
Note that the CI run for the exercise can be minimal.
PR-URL: #18846
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Matheus Marchini <matheus@sthima.com>
Reviewed-By: Jon Moss <me@jonathanmoss.me>
Reviewed-By: Gibson Fahnestock <gibfahn@gmail.com>
Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>1 parent 2a2f6be commit 58ecdd4
1 file changed
Lines changed: 4 additions & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -200,7 +200,10 @@ onboarding session. | |||
| 200 | 200 | * Collaborators are in alphabetical order by GitHub username. | |
| 201 | 201 | * Optionally, include your personal pronouns. | |
| 202 | 202 | * Label your pull request with the `doc` subsystem label. | |
| 203 | - * Run CI on your PR. | ||
| 203 | + * Run CI on the PR. Because the PR does not affect any code, use the | ||
| 204 | + `node-test-pull-request-lite` CI task. Alternatively, use the usual | ||
| 205 | + `node-test-pull-request` CI task and cancel it after the linter and one other | ||
| 206 | + subtask have passed. | ||
| 204 | 207 | * After one or two approvals, land the PR (PRs of this type do not need to wait | |
| 205 | 208 | for 48/72 hours to land). | |
| 206 | 209 | * Be sure to add the `PR-URL: <full-pr-url>` and appropriate `Reviewed-By:` | |
0 commit comments