From bc64cf2b5e8841515aa1ddeeb7bbf749a6b17d98 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 c7ad4eabdfa162..9bd94fc3679d20 100644 --- a/Makefile +++ b/Makefile @@ -735,10 +735,12 @@ bench-idle: $(NODE) benchmark/idle_clients.js & 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 @@ -761,12 +763,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