Skip to content

Commit 2a0fa47

Browse files
vsemozhetbyttargos
authored andcommitted
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 <[email protected]> Reviewed-By: Refael Ackermann <[email protected]> Reviewed-By: Richard Lau <[email protected]> Reviewed-By: João Reis <[email protected]> Reviewed-By: Jon Moss <[email protected]> Reviewed-By: Rich Trott <[email protected]> Reviewed-By: Anatoli Papirovski <[email protected]>
1 parent 4958501 commit 2a0fa47

File tree

1 file changed

+1
-3
lines changed

1 file changed

+1
-3
lines changed

doc/onboarding.md

+1-3
Original file line numberDiff line numberDiff line change
@@ -213,9 +213,7 @@ needs to be pointed out separately during the onboarding.
213213
* Optionally, include your personal pronouns.
214214
* Label your pull request with the `doc` and `notable-change` labels.
215215
* 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.
219217
* After one or two approvals, land the PR (PRs of this type do not need to wait
220218
for 48/72 hours to land).
221219
* Be sure to add the `PR-URL: <full-pr-url>` and appropriate `Reviewed-By:`

0 commit comments

Comments
 (0)