Skip to content

Commit 261ef1d

Browse files
TrottMylesBorins
authored andcommitted
doc: remove POST_STATUS_TO_PR from onboarding.md
POST_STATUS_TO_PR is checked/enabled by default so there is no longer a need to mention that it should be checked in onboarding.md. There is almost never a situation when it should be unchecked. PR-URL: #21042 Reviewed-By: Refael Ackermann <[email protected]> Reviewed-By: Jon Moss <[email protected]> Reviewed-By: Vse Mozhet Byt <[email protected]> Reviewed-By: Trivikram Kamat <[email protected]> Reviewed-By: Tobias Nießen <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Luigi Pinca <[email protected]>
1 parent 2edf172 commit 261ef1d

File tree

1 file changed

+1
-3
lines changed

1 file changed

+1
-3
lines changed

doc/onboarding.md

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -185,9 +185,7 @@ onboarding session.
185185
request containing the code you wish to test. For example, if the URL for
186186
the pull request is `https://github.com/nodejs/node/issues/7006`, then put
187187
`7006` in the `PR_ID`.
188-
* The remaining elements on the form are typically unchanged with the
189-
exception of `POST_STATUS_TO_PR`. Check that if you want a CI status
190-
indicator to be automatically inserted into the PR.
188+
* The remaining elements on the form are typically unchanged.
191189
* If you need help with something CI-related:
192190
* Use #node-dev (IRC) to talk to other Collaborators.
193191
* Use #node-build (IRC) to talk to the Build WG members who maintain the CI

0 commit comments

Comments
 (0)