From 188387aa220ca84503d638981915660340f63e67 Mon Sep 17 00:00:00 2001 From: Rich Trott Date: Tue, 16 Aug 2016 14:18:51 -0700 Subject: [PATCH] build: do not lint src dir for JS errors There are no JavaScript files in the `src` directory. It can be safely omitted from the JavaScript linting step. --- Makefile | 4 ++-- vcbuild.bat | 4 ++-- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/Makefile b/Makefile index fac19326946302..f3ff2d575bbcad 100644 --- a/Makefile +++ b/Makefile @@ -671,11 +671,11 @@ bench: bench-net bench-http bench-fs bench-tls bench-ci: bench jslint: - $(NODE) tools/jslint.js -J benchmark lib src test tools + $(NODE) tools/jslint.js -J benchmark lib test tools jslint-ci: $(NODE) tools/jslint.js $(PARALLEL_ARGS) -f tap -o test-eslint.tap \ - benchmark lib src test tools + benchmark lib test tools CPPLINT_EXCLUDE ?= CPPLINT_EXCLUDE += src/node_root_certs.h diff --git a/vcbuild.bat b/vcbuild.bat index 3d73ce8142d262..701127e4d2d2ce 100644 --- a/vcbuild.bat +++ b/vcbuild.bat @@ -351,12 +351,12 @@ if defined jslint_ci goto jslint-ci if not defined jslint goto exit if not exist tools\eslint\lib\eslint.js goto no-lint echo running jslint -%config%\node tools\jslint.js -J benchmark lib src test tools +%config%\node tools\jslint.js -J benchmark lib test tools goto exit :jslint-ci echo running jslint-ci -%config%\node tools\jslint.js -J -f tap -o test-eslint.tap benchmark lib src test tools +%config%\node tools\jslint.js -J -f tap -o test-eslint.tap benchmark lib test tools goto exit :no-lint