From e6be4b951a373605ec5e275885b7c1b6e57f9f82 Mon Sep 17 00:00:00 2001 From: "Sakthipriyan Vairamani (thefourtheye)" Date: Sat, 15 Apr 2017 13:53:46 +0530 Subject: [PATCH] build: make linter targets silent The linter targets are printing the commands they execute on screen. This patch reduces the noise by not printing the commands. PR-URL: https://github.com/nodejs/node/pull/12423 Reviewed-By: Ben Noordhuis Reviewed-By: Benjamin Gruenbaum Reviewed-By: Colin Ihrig Reviewed-By: Gibson Fahnestock Reviewed-By: James M Snell --- Makefile | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/Makefile b/Makefile index d0ba8cf8fd33fa..17dcfa06166378 100644 --- a/Makefile +++ b/Makefile @@ -809,10 +809,12 @@ bench: bench-net bench-http bench-fs bench-tls bench-ci: bench jslint: + @echo "Running JS linter..." $(NODE) tools/eslint/bin/eslint.js --cache --rulesdir=tools/eslint-rules \ benchmark lib test tools jslint-ci: + @echo "Running JS linter..." $(NODE) tools/jslint.js $(PARALLEL_ARGS) -f tap -o test-eslint.tap \ benchmark lib test tools @@ -836,12 +838,13 @@ CPPLINT_FILES = $(filter-out $(CPPLINT_EXCLUDE), $(wildcard \ )) cpplint: + @echo "Running C++ linter..." @$(PYTHON) tools/cpplint.py $(CPPLINT_FILES) @$(PYTHON) tools/check-imports.py ifneq ("","$(wildcard tools/eslint/lib/eslint.js)") lint: - EXIT_STATUS=0 ; \ + @EXIT_STATUS=0 ; \ $(MAKE) jslint || EXIT_STATUS=$$? ; \ $(MAKE) cpplint || EXIT_STATUS=$$? ; \ exit $$EXIT_STATUS