Commit 2875459
committed
build: run linter before running tests
PR-URL: #16284
Fixes: https://github.com/node/issues/16283
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Gibson Fahnestock <gibfahn@gmail.com>
Reviewed-By: Refael Ackermann <refack@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Anatoli Papirovski <apapirovski@mac.com>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: Franziska Hinkelmann <franziska.hinkelmann@gmail.com>
Reviewed-By: Richard Lau <riclau@uk.ibm.com>1 parent a70ef36 commit 2875459
1 file changed
Lines changed: 13 additions & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -213,15 +213,26 @@ test: all | |||
| 213 | 213 | $(MAKE) build-addons | |
| 214 | 214 | $(MAKE) build-addons-napi | |
| 215 | 215 | $(MAKE) doc-only | |
| 216 | + $(MAKE) lint | ||
| 216 | 217 | $(MAKE) cctest | |
| 217 | 218 | $(PYTHON) tools/test.py --mode=release -J \ | |
| 218 | 219 | $(CI_ASYNC_HOOKS) \ | |
| 219 | 220 | $(CI_JS_SUITES) \ | |
| 220 | 221 | $(CI_NATIVE_SUITES) \ | |
| 221 | 222 | doctool known_issues | |
| 222 | - $(MAKE) lint | ||
| 223 | 223 | endif | |
| 224 | 224 | ||
| 225 | + # For a quick test, does not run linter or build doc | ||
| 226 | + test-only: all | ||
| 227 | + $(MAKE) build-addons | ||
| 228 | + $(MAKE) build-addons-napi | ||
| 229 | + $(MAKE) cctest | ||
| 230 | + $(PYTHON) tools/test.py --mode=release -J \ | ||
| 231 | + $(CI_ASYNC_HOOKS) \ | ||
| 232 | + $(CI_JS_SUITES) \ | ||
| 233 | + $(CI_NATIVE_SUITES) \ | ||
| 234 | + known_issues | ||
| 235 | + | ||
| 225 | 236 | test-cov: all | |
| 226 | 237 | $(MAKE) build-addons | |
| 227 | 238 | $(MAKE) build-addons-napi | |
@@ -1107,6 +1118,7 @@ endif | |||
| 1107 | 1118 | test-gc \ | |
| 1108 | 1119 | test-gc-clean \ | |
| 1109 | 1120 | test-hash-seed \ | |
| 1121 | + test-only \ | ||
| 1110 | 1122 | test-v8 \ | |
| 1111 | 1123 | test-v8-all \ | |
| 1112 | 1124 | test-v8-benchmarks \ | |
0 commit comments