forked from facebook/react
-
Notifications
You must be signed in to change notification settings - Fork 1
/
.eslintrc
56 lines (49 loc) · 1.42 KB
/
.eslintrc
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
---
parser: babel-eslint
env:
browser: true
node: true
globals:
__DEV__: true
rules:
# ERRORS
space-before-blocks: 2
indent: [2, 2, indentSwitchCase: true]
brace-style: 2
space-after-keywords: 2
strict: 2
# We actually have a transform to support this and we fix this for bundled
# releases but not for the npm package, so enforce it strictly
comma-dangle: [2, never]
# Make this a warning for now. We do this in a few places so we might need to
# disable
no-unused-expressions: 2
eol-last: 2
dot-notation: 2
dot-location: [2, property]
consistent-return: 2
no-unused-vars: [2, args: none]
quotes: [2, single]
no-shadow: 2
no-multi-spaces: 2
# WARNINGS
# This is the only one that's hard to track since we don't lint just changes.
max-len: [1, 80]
# WISHLIST. One day...
# We'll need a custom version of this that does a subset of the whole rule.
# Otherwise this is just too noisy.
# valid-jsdoc: 1
# DISABLED. These aren't compatible with our style
# We use this for private/internal variables
no-underscore-dangle: 0
# We pass constructors around / access them from members
new-cap: 0
# We do this a lot.
no-use-before-define: 0
# We do this in a few places to align values
key-spacing: 0
# It's nice to be able to leave catch blocks empty
no-empty: 0
# BROKEN. We'd like to turn these back on.
# causes a ton of noise, eslint is too picky?
block-scoped-var: 0