Commit 0d19853
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 <r.m.graham@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: James M Snell <jasnell@gmail.com>1 parent 4068d64 commit 0d19853
1 file changed
Lines changed: 3 additions & 3 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -115,9 +115,9 @@ v8: | |||
| 115 | 115 | $(MAKE) -C deps/v8 $(V8_ARCH) $(V8_BUILD_OPTIONS) | |
| 116 | 116 | ||
| 117 | 117 | 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 | ||
| 121 | 121 | ||
| 122 | 122 | test-parallel: all | |
| 123 | 123 | $(PYTHON) tools/test.py --mode=release parallel -J | |
0 commit comments