Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly
Appearance settings

Commit 58ecdd4

Browse filesBrowse files
TrottMylesBorins
Trott
authored and
MylesBorins
committed
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
Copy full SHA for 58ecdd4

File tree

Expand file treeCollapse file tree

1 file changed

+4
-1
lines changed
Open diff view settings
Filter options
Expand file treeCollapse file tree

1 file changed

+4
-1
lines changed
Open diff view settings
Collapse file

‎doc/onboarding.md‎

Copy file name to clipboardExpand all lines: doc/onboarding.md
+4-1Lines changed: 4 additions & 1 deletion
  • Display the source diff
  • Display the rich diff
Original file line numberDiff line numberDiff line change
@@ -200,7 +200,10 @@ onboarding session.
200200
* Collaborators are in alphabetical order by GitHub username.
201201
* Optionally, include your personal pronouns.
202202
* 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.
204207
* After one or two approvals, land the PR (PRs of this type do not need to wait
205208
for 48/72 hours to land).
206209
* Be sure to add the `PR-URL: <full-pr-url>` and appropriate `Reviewed-By:`

0 commit comments

Comments
0 (0)
Morty Proxy This is a proxified and sanitized view of the page, visit original site.