Skip to content

Commit a398516

Browse files
Trottaddaleax
authored andcommitted
doc: remove test-npm from general build doc
`make test-npm` is not particularly robust (currently fails on Linux and Windows, reportedly) and results in a fair number of requests for help from people new to the project. It is used when upgrading npm in core, which only a small number of mostly-predefined people do. Remove it from the general build doc. Refs: #12836 PR-URL: #12840 Reviewed-By: Anna Henningsen <[email protected]> Reviewed-By: Jeremiah Senkpiel <[email protected]> Reviewed-By: Daniel Bevenius <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Colin Ihrig <[email protected]>
1 parent a710e44 commit a398516

File tree

1 file changed

+0
-9
lines changed

1 file changed

+0
-9
lines changed

BUILDING.md

-9
Original file line numberDiff line numberDiff line change
@@ -123,15 +123,6 @@ To run the tests:
123123
$ make test
124124
```
125125

126-
To run the npm test suite:
127-
128-
*note: to run the suite on node v4 or earlier you must first*
129-
*run `make install`*
130-
131-
```console
132-
$ make test-npm
133-
```
134-
135126
To build the documentation:
136127

137128
This will build Node.js first (if necessary) and then use it to build the docs:

0 commit comments

Comments
 (0)