Skip to content

Commit 0d19853

Browse files
jbergstroemFishrock123
authored andcommitted
build: use shorthand lint target from test
Instead of invoking jslint/cpplint from the test target, call on the generic lint instead since it checks if eslint exists. Since our tarballs lacks eslint we now get a more graceful exit from `make test` instead of a traceback from jslint. PR-URL: #6406 Fixes: #6408 Reviewed-By: Ryan Graham <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Anna Henningsen <[email protected]> Reviewed-By: James M Snell <[email protected]>
1 parent 4068d64 commit 0d19853

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

Makefile

+3-3
Original file line numberDiff line numberDiff line change
@@ -115,9 +115,9 @@ v8:
115115
$(MAKE) -C deps/v8 $(V8_ARCH) $(V8_BUILD_OPTIONS)
116116

117117
test: | cctest # Depends on 'all'.
118-
$(PYTHON) tools/test.py --mode=release doctool message parallel sequential -J
119-
$(MAKE) jslint
120-
$(MAKE) cpplint
118+
$(PYTHON) tools/test.py --mode=release -J \
119+
doctool message parallel sequential
120+
$(MAKE) lint
121121

122122
test-parallel: all
123123
$(PYTHON) tools/test.py --mode=release parallel -J

0 commit comments

Comments
 (0)