Skip to content

Commit d85840d

Browse files
seishunMylesBorins
authored andcommitted
doc: require CI status indicator in PRs
Commits are often landed despite failing on one or more CI platforms. Having a CI status indicator in the PR should make this less likely to happen. PR-URL: #17151 Reviewed-By: Michaël Zasso <[email protected]> Reviewed-By: Joyee Cheung <[email protected]> Reviewed-By: Gibson Fahnestock <[email protected]> Reviewed-By: Jon Moss <[email protected]> Reviewed-By: Tobias Nießen <[email protected]>
1 parent ffc1444 commit d85840d

File tree

1 file changed

+1
-0
lines changed

1 file changed

+1
-0
lines changed

COLLABORATOR_GUIDE.md

+1
Original file line numberDiff line numberDiff line change
@@ -142,6 +142,7 @@ test should *fail* before the change, and *pass* after the change.
142142
All pull requests that modify executable code should be subjected to
143143
continuous integration tests on the
144144
[project CI server](https://ci.nodejs.org/).
145+
The pull request should have a CI status indicator if possible.
145146

146147
#### Useful CI Jobs
147148

0 commit comments

Comments
 (0)