Skip to content

Commit b44e789

Browse files
Jonathan DarlingFishrock123
Jonathan Darling
authored andcommitted
doc: standardizing on make -j4
Standardizes docs to use -j4 instead of -j8 as it appears to be the most inclusive recommendation based on discussion in #9961. PR-URL: #9961 Reviewed-By: Jeremiah Senkpiel <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Gibson Fahnestock <[email protected]> Reviewed-By: Ben Noordhuis <[email protected]> Reviewed-By: Stephen Belanger <[email protected]>
1 parent ff8fdb1 commit b44e789

4 files changed

+5
-5
lines changed

.github/PULL_REQUEST_TEMPLATE.md

+1-1
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@ Contributors guide: https://github.com/nodejs/node/blob/master/CONTRIBUTING.md
99
##### Checklist
1010
<!-- Remove items that do not apply. For completed items, change [ ] to [x]. -->
1111

12-
- [ ] `make -j8 test` (UNIX), or `vcbuild test nosign` (Windows) passes
12+
- [ ] `make -j4 test` (UNIX), or `vcbuild test nosign` (Windows) passes
1313
- [ ] tests and/or benchmarks are included
1414
- [ ] documentation is changed or added
1515
- [ ] commit message follows commit guidelines

CONTRIBUTING.md

+2-2
Original file line numberDiff line numberDiff line change
@@ -156,7 +156,7 @@ to see how they should be structured can also help.
156156
To run the tests on Unix / OS X:
157157

158158
```text
159-
$ ./configure && make -j8 test
159+
$ ./configure && make -j4 test
160160
```
161161

162162
Windows:
@@ -189,7 +189,7 @@ You can run tests directly with node:
189189
$ ./node ./test/parallel/test-stream2-transform.js
190190
```
191191

192-
Remember to recompile with `make -j8` in between test runs if you change
192+
Remember to recompile with `make -j4` in between test runs if you change
193193
core modules.
194194

195195
### Step 6: Push

doc/guides/building-node-with-ninja.md

+1-1
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@ ninja: Entering directory `out/Release`
1616

1717
The bottom line will change while building, showing the progress as `[finished/total]` build steps.
1818
This is useful output that `make` does not produce and is one of the benefits of using Ninja.
19-
Also, Ninja will likely compile much faster than even `make -j8` (or `-j<number of processor threads on your machine>`).
19+
Also, Ninja will likely compile much faster than even `make -j4` (or `-j<number of processor threads on your machine>`).
2020

2121
## Considerations
2222

doc/onboarding-extras.md

+1-1
Original file line numberDiff line numberDiff line change
@@ -71,7 +71,7 @@ Please use these when possible / appropriate
7171
* major vs. everything else: run last versions tests against this version, if they pass, **probably** minor or patch
7272
* A breaking change helper ([full source](https://gist.github.com/chrisdickinson/ba532fa0e4e243fb7b44)):
7373
```sh
74-
git checkout $(git show -s --pretty='%T' $(git show-ref -d $(git describe --abbrev=0) | tail -n1 | awk '{print $1}')) -- test; make -j8 test
74+
git checkout $(git show -s --pretty='%T' $(git show-ref -d $(git describe --abbrev=0) | tail -n1 | awk '{print $1}')) -- test; make -j4 test
7575
```
7676

7777

0 commit comments

Comments
 (0)