From 75a970bac373358f8215cb1da1195872f00beea2 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Jo=C3=A3o=20Reis?= Date: Mon, 26 Sep 2016 12:54:58 +0100 Subject: [PATCH] win,tools: ignore linting for line breaks Line breaks on Windows should be CRLF, but Node also supports LF. Hence, do not check line breaks on Windows, when running vcbuild jslint. Fixes: https://github.com/nodejs/node/issues/6912 PR-URL: https://github.com/nodejs/node/pull/8785 Reviewed-By: James M Snell Reviewed-By: Bartosz Sosnowski Reviewed-By: Ben Noordhuis --- vcbuild.bat | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/vcbuild.bat b/vcbuild.bat index 93de090fe9..01ca4b9b3a 100644 --- a/vcbuild.bat +++ b/vcbuild.bat @@ -345,7 +345,7 @@ 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\eslint\bin\eslint.js --cache --rulesdir=tools\eslint-rules benchmark lib test tools +%config%\node tools\eslint\bin\eslint.js --cache --rule "linebreak-style: 0" --rulesdir=tools\eslint-rules benchmark lib test tools goto exit :jslint-ci