Skip to content

Commit 1c578b7

Browse files
Trottmarco-ippolito
authored andcommitted
test: revise pull request guide text about code
Fixes: nodejs#45502 PR-URL: nodejs#45519 Reviewed-By: Chengzhong Wu <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Tobias Nießen <[email protected]>
1 parent bd960bf commit 1c578b7

File tree

1 file changed

+6
-6
lines changed

1 file changed

+6
-6
lines changed

doc/contributing/pull-requests.md

+6-6
Original file line numberDiff line numberDiff line change
@@ -107,13 +107,13 @@ git checkout -b my-branch -t upstream/HEAD
107107

108108
### Step 3: Code
109109

110-
The vast majority of pull requests opened against the `nodejs/node`
111-
repository includes changes to one or more of the following:
110+
Pull requests in Node.js typically involve changes to
111+
one or more of a few places in the repository.
112112

113-
* the C/C++ code contained in the `src` directory
114-
* the JavaScript code contained in the `lib` directory
115-
* the documentation in `doc/api`
116-
* tests within the `test` directory.
113+
* C/C++ code contained in the `src` directory
114+
* JavaScript code contained in the `lib` directory
115+
* Documentation in `doc/api`
116+
* Tests within the `test` directory
117117

118118
If you are modifying code, please be sure to run `make lint` (or
119119
`vcbuild.bat lint` on Windows) to ensure that the changes follow the Node.js

0 commit comments

Comments
 (0)