Skip to content

Commit e6be4b9

Browse files
thefourtheyeevanlucas
authored andcommitted
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: #12423 Reviewed-By: Ben Noordhuis <[email protected]> Reviewed-By: Benjamin Gruenbaum <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Gibson Fahnestock <[email protected]> Reviewed-By: James M Snell <[email protected]>
1 parent 917534d commit e6be4b9

File tree

1 file changed

+4
-1
lines changed

1 file changed

+4
-1
lines changed

Makefile

+4-1
Original file line numberDiff line numberDiff line change
@@ -809,10 +809,12 @@ bench: bench-net bench-http bench-fs bench-tls
809809
bench-ci: bench
810810

811811
jslint:
812+
@echo "Running JS linter..."
812813
$(NODE) tools/eslint/bin/eslint.js --cache --rulesdir=tools/eslint-rules \
813814
benchmark lib test tools
814815

815816
jslint-ci:
817+
@echo "Running JS linter..."
816818
$(NODE) tools/jslint.js $(PARALLEL_ARGS) -f tap -o test-eslint.tap \
817819
benchmark lib test tools
818820

@@ -836,12 +838,13 @@ CPPLINT_FILES = $(filter-out $(CPPLINT_EXCLUDE), $(wildcard \
836838
))
837839

838840
cpplint:
841+
@echo "Running C++ linter..."
839842
@$(PYTHON) tools/cpplint.py $(CPPLINT_FILES)
840843
@$(PYTHON) tools/check-imports.py
841844

842845
ifneq ("","$(wildcard tools/eslint/lib/eslint.js)")
843846
lint:
844-
EXIT_STATUS=0 ; \
847+
@EXIT_STATUS=0 ; \
845848
$(MAKE) jslint || EXIT_STATUS=$$? ; \
846849
$(MAKE) cpplint || EXIT_STATUS=$$? ; \
847850
exit $$EXIT_STATUS

0 commit comments

Comments
 (0)