Skip to content

Commit d572f60

Browse files
Trotttargos
authored andcommitted
build: remove unnecessary Makefile output
Remove unnecessary @echo commands from Makefile. These were originally comments but were changed to @echo in 6bc43ae. They aren't terribly useful so let's remove them. PR-URL: #23129 Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Sakthipriyan Vairamani <[email protected]> Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Refael Ackermann <[email protected]> Reviewed-By: James M Snell <[email protected]>
1 parent 7d7dc16 commit d572f60

File tree

1 file changed

+0
-6
lines changed

1 file changed

+0
-6
lines changed

Makefile

-6
Original file line numberDiff line numberDiff line change
@@ -279,26 +279,20 @@ jstest: build-addons build-addons-napi ## Runs addon tests and JS tests
279279
# This does not run tests of third-party libraries inside deps.
280280
test: all ## Runs default tests, linters, and builds docs.
281281
$(MAKE) -s test-doc
282-
@echo "Build the addons before running the tests so the test results"
283-
@echo "can be displayed together"
284282
$(MAKE) -s build-addons
285283
$(MAKE) -s build-addons-napi
286284
$(MAKE) -s cctest
287285
$(MAKE) -s jstest
288286

289287
.PHONY: test-only
290288
test-only: all ## For a quick test, does not run linter or build docs.
291-
@echo "Build the addons before running the tests so the test results"
292-
@echo "can be displayed together"
293289
$(MAKE) build-addons
294290
$(MAKE) build-addons-napi
295291
$(MAKE) cctest
296292
$(MAKE) jstest
297293

298294
# Used by `make coverage-test`
299295
test-cov: all
300-
@echo "Build the addons before running the tests so the test results"
301-
@echo "can be displayed together"
302296
$(MAKE) build-addons
303297
$(MAKE) build-addons-napi
304298
# $(MAKE) cctest

0 commit comments

Comments
 (0)