From f4c3456610361928145f51e84879522fceb23d78 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 MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit There are no JavaScript files in the `src` directory. It can be safely omitted from the JavaScript linting step. PR-URL: https://github.com/nodejs/node/pull/8128 Reviewed-By: Colin Ihrig Reviewed-By: Johan Bergström Reviewed-By: Roman Reiss Reviewed-By: James M Snell --- Makefile | 4 ++-- vcbuild.bat | 4 ++-- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/Makefile b/Makefile index a28dc3176115a6..a8870a55429599 100644 --- a/Makefile +++ b/Makefile @@ -679,11 +679,11 @@ bench-idle: $(NODE) benchmark/idle_clients.js & 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 195215c8902269..aa6d82facbc8b3 100644 --- a/vcbuild.bat +++ b/vcbuild.bat @@ -348,12 +348,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