Commit 2a0fa47
doc: do not advise to cancel full CI on onboarding
Refs: #21938 (comment)
and discussion below the comment.
PR-URL: #21977
Reviewed-By: Matheus Marchini <matheus@sthima.com>
Reviewed-By: Refael Ackermann <refack@gmail.com>
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: João Reis <reis@janeasystems.com>
Reviewed-By: Jon Moss <me@jonathanmoss.me>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Anatoli Papirovski <apapirovski@mac.com>1 parent 4958501 commit 2a0fa47
1 file changed
Lines changed: 1 addition & 3 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -213,9 +213,7 @@ needs to be pointed out separately during the onboarding. | |||
| 213 | 213 | * Optionally, include your personal pronouns. | |
| 214 | 214 | * Label your pull request with the `doc` and `notable-change` labels. | |
| 215 | 215 | * Run CI on the PR. Because the PR does not affect any code, use the | |
| 216 | - `node-test-pull-request-lite-pipeline` CI task. Alternatively, use the usual | ||
| 217 | - `node-test-pull-request` CI task and cancel it after the linter and one other | ||
| 218 | - subtask have passed. | ||
| 216 | + `node-test-pull-request-lite-pipeline` CI task. | ||
| 219 | 217 | * After one or two approvals, land the PR (PRs of this type do not need to wait | |
| 220 | 218 | for 48/72 hours to land). | |
| 221 | 219 | * Be sure to add the `PR-URL: <full-pr-url>` and appropriate `Reviewed-By:` | |
0 commit comments