FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

tools: consistent .eslintrc formatting · nodejs/node@5a2ce36 · GitHub

/ node Public

Commit 5a2ce36

Browse files
silverwind
authored and
evanlucas
committed
tools: consistent .eslintrc formatting
All quotes in .eslintrc were unnecessary and inconsistently placed across the file. Additionally, format the globals to be consistent with the style of whitespace and sorted them alphabetically. PR-URL: #7691 Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Rich Trott <rtrott@gmail.com> Reviewed-By: Michaël Zasso <mic.besace@gmail.com>
1 parent 1d07d29 commit 5a2ce36

2 files changed

Lines changed: 34 additions & 34 deletions

File tree

‎.eslintrc‎

Lines changed: 33 additions & 33 deletions
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@ env:
55
rules:
66
# Possible Errors
77
# http://eslint.org/docs/rules/#possible-errors
8-
comma-dangle: [2, "only-multiline"]
8+
comma-dangle: [2, only-multiline]
99
no-control-regex: 2
1010
no-debugger: 2
1111
no-dupe-args: 2
@@ -14,7 +14,7 @@ rules:
1414
no-empty-character-class: 2
1515
no-ex-assign: 2
1616
no-extra-boolean-cast: 2
17-
no-extra-parens: [2, "functions"]
17+
no-extra-parens: [2, functions]
1818
no-extra-semi: 2
1919
no-func-assign: 2
2020
no-invalid-regexp: 2
@@ -38,47 +38,47 @@ rules:
3838

3939
# Strict Mode
4040
# http://eslint.org/docs/rules/#strict-mode
41-
strict: [2, "global"]
41+
strict: [2, global]
4242

4343
# Variables
4444
# http://eslint.org/docs/rules/#variables
4545
no-delete-var: 2
4646
no-undef: 2
47-
no-unused-vars: [2, {"args": "none"}]
47+
no-unused-vars: [2, {args: none}]
4848

4949
# Node.js and CommonJS
5050
# http://eslint.org/docs/rules/#nodejs-and-commonjs
5151
no-mixed-requires: 2
5252
no-new-require: 2
5353
no-path-concat: 2
54-
no-restricted-modules: [2, "sys", "_linklist"]
54+
no-restricted-modules: [2, sys, _linklist]
5555

5656
# Stylistic Issues
5757
# http://eslint.org/docs/rules/#stylistic-issues
58-
brace-style: [2, "1tbs", {allowSingleLine: true}]
58+
brace-style: [2, 1tbs, {allowSingleLine: true}]
5959
comma-spacing: 2
6060
eol-last: 2
6161
indent: [2, 2, {SwitchCase: 1}]
62-
key-spacing: [2, {mode: "minimum"}]
62+
key-spacing: [2, {mode: minimum}]
6363
keyword-spacing: 2
64-
linebreak-style: [2, "unix"]
64+
linebreak-style: [2, unix]
6565
max-len: [2, 80, 2]
6666
new-parens: 2
6767
no-mixed-spaces-and-tabs: 2
6868
no-multiple-empty-lines: [2, {max: 2}]
6969
no-trailing-spaces: 2
70-
quotes: [2, "single", "avoid-escape"]
70+
quotes: [2, single, avoid-escape]
7171
semi: 2
72-
space-before-blocks: [2, "always"]
73-
space-before-function-paren: [2, "never"]
74-
space-in-parens: [2, "never"]
72+
space-before-blocks: [2, always]
73+
space-before-function-paren: [2, never]
74+
space-in-parens: [2, never]
7575
space-infix-ops: 2
7676
space-unary-ops: 2
7777

7878
# ECMAScript 6
7979
# http://eslint.org/docs/rules/#ecmascript-6
80-
arrow-parens: [2, "always"]
81-
arrow-spacing: [2, {"before": true, "after": true}]
80+
arrow-parens: [2, always]
81+
arrow-spacing: [2, {before: true, after: true}]
8282
constructor-super: 2
8383
no-class-assign: 2
8484
no-confusing-arrow: 2
@@ -93,27 +93,27 @@ rules:
9393
align-function-arguments: 2
9494
align-multiline-assignment: 2
9595
assert-fail-single-argument: 2
96-
new-with-error: [2, "Error", "RangeError", "TypeError", "SyntaxError", "ReferenceError"]
96+
new-with-error: [2, Error, RangeError, TypeError, SyntaxError, ReferenceError]
9797
no-deepEqual: 2
9898
no-definegetter-definesetter: 2
9999

100100
# Global scoped method and vars
101101
globals:
102-
DTRACE_HTTP_CLIENT_REQUEST : false
103-
LTTNG_HTTP_CLIENT_REQUEST : false
104-
COUNTER_HTTP_CLIENT_REQUEST : false
105-
DTRACE_HTTP_CLIENT_RESPONSE : false
106-
LTTNG_HTTP_CLIENT_RESPONSE : false
107-
COUNTER_HTTP_CLIENT_RESPONSE : false
108-
DTRACE_HTTP_SERVER_REQUEST : false
109-
LTTNG_HTTP_SERVER_REQUEST : false
110-
COUNTER_HTTP_SERVER_REQUEST : false
111-
DTRACE_HTTP_SERVER_RESPONSE : false
112-
LTTNG_HTTP_SERVER_RESPONSE : false
113-
COUNTER_HTTP_SERVER_RESPONSE : false
114-
DTRACE_NET_STREAM_END : false
115-
LTTNG_NET_STREAM_END : false
116-
COUNTER_NET_SERVER_CONNECTION_CLOSE : false
117-
DTRACE_NET_SERVER_CONNECTION : false
118-
LTTNG_NET_SERVER_CONNECTION : false
119-
COUNTER_NET_SERVER_CONNECTION : false
102+
COUNTER_HTTP_CLIENT_REQUEST: false
103+
COUNTER_HTTP_CLIENT_RESPONSE: false
104+
COUNTER_HTTP_SERVER_REQUEST: false
105+
COUNTER_HTTP_SERVER_RESPONSE: false
106+
COUNTER_NET_SERVER_CONNECTION: false
107+
COUNTER_NET_SERVER_CONNECTION_CLOSE: false
108+
DTRACE_HTTP_CLIENT_REQUEST: false
109+
DTRACE_HTTP_CLIENT_RESPONSE: false
110+
DTRACE_HTTP_SERVER_REQUEST: false
111+
DTRACE_HTTP_SERVER_RESPONSE: false
112+
DTRACE_NET_SERVER_CONNECTION: false
113+
DTRACE_NET_STREAM_END: false
114+
LTTNG_HTTP_CLIENT_REQUEST: false
115+
LTTNG_HTTP_CLIENT_RESPONSE: false
116+
LTTNG_HTTP_SERVER_REQUEST: false
117+
LTTNG_HTTP_SERVER_RESPONSE: false
118+
LTTNG_NET_SERVER_CONNECTION: false
119+
LTTNG_NET_STREAM_END: false

‎test/.eslintrc‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,4 +2,4 @@
22

33
rules:
44
## common module is mandatory in tests
5-
required-modules: [2, "common"]
5+
required-modules: [2, common]

0 commit comments

Comments
 (0)
Back | FazBrowse Home | New Git URL