Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Commit 578d80b

Browse filesBrowse files
danbevMylesBorins
authored andcommitted
build: prevent echoing of recipes for test target
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: #17010 Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>
1 parent a05c49c commit 578d80b
Copy full SHA for 578d80b

File tree

Expand file treeCollapse file tree

1 file changed

+5
-5
lines changed
Open diff view settings
Filter options
Expand file treeCollapse file tree

1 file changed

+5
-5
lines changed
Open diff view settings
Collapse file

‎Makefile‎

Copy file name to clipboardExpand all lines: Makefile
+5-5Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -210,11 +210,11 @@ test: all
210210
$(MAKE) cctest
211211
else
212212
test: all
213-
$(MAKE) build-addons
214-
$(MAKE) build-addons-napi
215-
$(MAKE) doc-only
216-
$(MAKE) lint
217-
$(MAKE) cctest
213+
$(MAKE) -s build-addons
214+
$(MAKE) -s build-addons-napi
215+
$(MAKE) -s doc-only
216+
$(MAKE) -s lint
217+
$(MAKE) -s cctest
218218
$(PYTHON) tools/test.py --mode=release -J \
219219
$(CI_ASYNC_HOOKS) \
220220
$(CI_JS_SUITES) \

0 commit comments

Comments
0 (0)
Morty Proxy This is a proxified and sanitized view of the page, visit original site.