1berry/nodejs
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

build: prevent echoing of recipes for test target

Browse Source 
Currenlty the test target will echo additional information that might
not be that useful, for example:
make doc-only
make[1]: Nothing to be done for `doc-only'.
make lint
Running JS linter...
Running C++ linter...
Total errors found: 0
make[2]: Nothing to be done for `lint-md'.
Running C++ linter on addon docs...
Total errors found: 0
make cctest

This commit suggests reducing this to:
make -s doc-only
make -s lint
Running JS linter...
Running C++ linter...
Running C++ linter on addon docs...
Total errors found: 0
make -s cctest

PR-URL: https://github.com/nodejs/node/pull/17010
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>
This commit is contained in:
Daniel Bevenius 2017-11-14 08:41:10 +01:00
parent 1b093cb93d
commit a0b1e2eca6
1 changed files with 5 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
Makefile
View File

@ -210,11 +210,11 @@ test: all
$(MAKE) cctest $(MAKE) cctest
else else
test: all test: all
$(MAKE) build-addons $(MAKE) -s build-addons
$(MAKE) build-addons-napi $(MAKE) -s build-addons-napi
$(MAKE) doc-only $(MAKE) -s doc-only
$(MAKE) lint $(MAKE) -s lint
$(MAKE) cctest $(MAKE) -s cctest
$(PYTHON) tools/test.py --mode=release -J \ $(PYTHON) tools/test.py --mode=release -J \
$(CI_ASYNC_HOOKS) \ $(CI_ASYNC_HOOKS) \
$(CI_JS_SUITES) \ $(CI_JS_SUITES) \