Skip to content
This repository has been archived by the owner on Oct 15, 2020. It is now read-only.

Commit

Permalink
tool: modified eslint rule for line-breaks
Browse files Browse the repository at this point in the history
Recently linebreak-style rule was added in jslint which expects
linebreaks to be unix style i.e. `LF` by default. However windows
has linebreak `CRLF` (`git config auto.crlf=true`) causing this rule
to fail for all `*.js` files present in repo. I have disabled the rule
for now and opened nodejs/node#6912 to get the right rule
for windows user.

PR-URL: #73
Reviewed-By: Jianchun Xu <Jianchun.Xu@microsoft.com>
  • Loading branch information
kunalspathak committed May 25, 2016
1 parent 2048b2c commit 8d11795
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion .eslintrc
Original file line number Diff line number Diff line change
Expand Up @@ -60,7 +60,7 @@ rules:
indent: [2, 2, {SwitchCase: 1}]
key-spacing: [2, {mode: "minimum"}]
keyword-spacing: 2
linebreak-style: [2, "unix"]
linebreak-style: [1, "unix"]
max-len: [2, 80, 2]
new-parens: 2
no-mixed-spaces-and-tabs: 2
Expand Down

0 comments on commit 8d11795

Please sign in to comment.