forked from firefox-devtools/devtools-core
-
Notifications
You must be signed in to change notification settings - Fork 0
/
.eslintrc
613 lines (613 loc) · 21.6 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
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
244
245
246
247
248
249
250
251
252
253
254
255
256
257
258
259
260
261
262
263
264
265
266
267
268
269
270
271
272
273
274
275
276
277
278
279
280
281
282
283
284
285
286
287
288
289
290
291
292
293
294
295
296
297
298
299
300
301
302
303
304
305
306
307
308
309
310
311
312
313
314
315
316
317
318
319
320
321
322
323
324
325
326
327
328
329
330
331
332
333
334
335
336
337
338
339
340
341
342
343
344
345
346
347
348
349
350
351
352
353
354
355
356
357
358
359
360
361
362
363
364
365
366
367
368
369
370
371
372
373
374
375
376
377
378
379
380
381
382
383
384
385
386
387
388
389
390
391
392
393
394
395
396
397
398
399
400
401
402
403
404
405
406
407
408
409
410
411
412
413
414
415
416
417
418
419
420
421
422
423
424
425
426
427
428
429
430
431
432
433
434
435
436
437
438
439
440
441
442
443
444
445
446
447
448
449
450
451
452
453
454
455
456
457
458
459
460
461
462
463
464
465
466
467
468
469
470
471
472
473
474
475
476
477
478
479
480
481
482
483
484
485
486
487
488
489
490
491
492
493
494
495
496
497
498
499
500
501
502
503
504
505
506
507
508
509
510
511
512
513
514
515
516
517
518
519
520
521
522
523
524
525
526
527
528
529
530
531
532
533
534
535
536
537
538
539
540
541
542
543
544
545
546
547
548
549
550
551
552
553
554
555
556
557
558
559
560
561
562
563
564
565
566
567
568
569
570
571
572
573
574
575
576
577
578
579
580
581
582
583
584
585
586
587
588
589
590
591
592
593
594
595
596
597
598
599
600
601
602
603
604
605
606
607
608
609
610
611
612
613
{
"parser": "babel-eslint",
"plugins": [
"react",
"mozilla",
"flowtype",
"babel",
"jest"
],
"globals": {
"atob": true,
"btoa": true,
"Cc": true,
"Ci": true,
"Components": true,
"console": true,
"Cr": true,
"Cu": true,
"devtools": true,
"dump": true,
"EventEmitter": true,
"exports": true,
"isWorker": true,
"loader": true,
"module": true,
"reportError": true,
"require": true,
"Services": true,
"Task": true,
"XPCNativeWrapper": true,
"XPCOMUtils": true,
"_Iterator": true,
"__dirname": true,
"process": true,
"expect": true,
"global": true,
"L10N": true
},
"env": {
"es6": true,
"browser": true,
"mocha": true
},
"rules": {
// These are the rules that have been configured so far to match the
// devtools coding style.
// Rules from the mozilla plugin
"mozilla/no-aArgs": "warn",
"mozilla/no-cpows-in-tests": "error",
"mozilla/no-single-arg-cu-import": "error",
// See bug 1224289.
"mozilla/reject-importGlobalProperties": "error",
// devtools/shared/platform is special; see the README.md in that
// directory for details. We reject requires using explicit
// subdirectories of this directory.
"mozilla/reject-some-requires": [
"error",
"^devtools/shared/platform/(chome|content)/"
],
"mozilla/var-only-at-top-level": "warn",
// Rules from the React plugin
"react/display-name": "error",
"react/no-danger": "error",
"react/no-did-mount-set-state": "error",
"react/no-did-update-set-state": "error",
"react/no-direct-mutation-state": "error",
"react/no-unknown-property": "error",
"react/prefer-es6-class": [
"warn",
"always"
],
"react/prop-types": "error",
"react/sort-comp": [
"error",
{
"order": [
"lifecycle",
"everything-else",
"render"
],
"groups": {
"lifecycle": [
"displayName",
"propTypes",
"contextTypes",
"childContextTypes",
"mixins",
"statics",
"defaultProps",
"constructor",
"getDefaultProps",
"getInitialState",
"state",
"getChildContext",
"componentWillMount",
"componentDidMount",
"componentWillReceiveProps",
"shouldComponentUpdate",
"componentWillUpdate",
"componentDidUpdate",
"componentWillUnmount"
]
}
}
],
"flowtype/define-flow-type": 1,
"flowtype/use-flow-type": 1,
// Disallow using variables outside the blocks they are defined (especially
// since only let and const are used, see "no-var").
"block-scoped-var": "error",
// Enforce one true brace style (opening brace on the same line) and avoid
// start and end braces on the same line.
"brace-style": [
"error",
"1tbs",
{
"allowSingleLine": false
}
],
// Require camel case names
"camelcase": "error",
// Allow trailing commas for easy list extension. Having them does not
// impair readability, but also not required either.
"comma-dangle": "off",
// Enforce spacing before and after comma
"comma-spacing": [
"error",
{
"before": false,
"after": true
}
],
// Enforce one true comma style.
"comma-style": [
"error",
"last"
],
// Warn about cyclomatic complexity in functions.
"complexity": [
"error",
35
],
// Require return statements to either always or never specify values.
"consistent-return": "error",
// Don't warn for inconsistent naming when capturing this (not so important
// with auto-binding fat arrow functions).
"consistent-this": "off",
// Enforce curly brace conventions for all control statements.
"curly": "error",
// Don't require a default case in switch statements. Avoid being forced to
// add a bogus default when you know all possible cases are handled.
"default-case": "off",
// Enforce dots on the next line with property name.
"dot-location": [
"error",
"property"
],
// Encourage the use of dot notation whenever possible.
"dot-notation": "error",
// Enforce newline at the end of file, with no multiple empty lines.
"eol-last": "error",
// Allow using == instead of ===, in the interest of landing something since
// the devtools codebase is split on convention here.
"eqeqeq": "off",
// Don't require function expressions to have a name.
// This makes the code more verbose and hard to read. Our engine already
// does a fantastic job assigning a name to the function, which includes
// the enclosing function name, and worst case you have a line number that
// you can just look up.
"func-names": "off",
// Allow use of function declarations and expressions.
"func-style": "off",
// Deprecated, will be removed in 1.0.
"generator-star": "off",
// Enforce the spacing around the * in generator functions.
"generator-star-spacing": [
"error",
"after"
],
// Deprecated, will be removed in 1.0.
"global-strict": "off",
// Only useful in a node environment.
"handle-callback-err": "off",
// Tab width.
"indent": [
"error",
2,
{
"SwitchCase": 1,
"ArrayExpression": "first",
"ObjectExpression": "first"
}
],
// Enforces spacing between keys and values in object literal properties.
"key-spacing": [
"error",
{
"beforeColon": false,
"afterColon": true
}
],
// Enforces unix style line breaks.
"linebreak-style": [
"error",
"unix"
],
// Don't enforce the maximum depth that blocks can be nested. The complexity
// rule is a better rule to check this.
"max-depth": "off",
// Maximum length of a line.
"max-len": [
"error",
90,
2,
{
"ignoreUrls": true,
"ignorePattern": "data:image\/|\\s*require\\s*\\(|^\\s*loader\\.lazy|-\\*-"
}
],
// Maximum depth callbacks can be nested.
"max-nested-callbacks": [
"error",
3
],
// Don't limit the number of parameters that can be used in a function.
"max-params": "off",
// Don't limit the maximum number of statement allowed in a function. We
// already have the complexity rule that's a better measurement.
"max-statements": "off",
// Require a capital letter for constructors, only check if all new
// operators are followed by a capital letter. Don't warn when capitalized
// functions are used without the new operator.
"new-cap": [
"error",
{
"capIsNew": false
}
],
// Disallow the omission of parentheses when invoking a constructor with no
// arguments.
"new-parens": "error",
// Disallow use of the Array constructor.
"no-array-constructor": "error",
// Allow use of bitwise operators.
"no-bitwise": "off",
// Disallow use of arguments.caller or arguments.callee.
"no-caller": "error",
// Disallow the catch clause parameter name being the same as a variable in
// the outer scope, to avoid confusion.
"no-catch-shadow": "error",
// Deprecated, will be removed in 1.0.
"no-comma-dangle": "off",
// Disallow assignment in conditional expressions.
"no-cond-assign": "error",
// Allow using the console API.
"no-console": "off",
// Allow using constant expressions in conditions like while (true)
"no-constant-condition": "off",
// Allow use of the continue statement.
"no-continue": "off",
// Disallow control characters in regular expressions.
"no-control-regex": "error",
// Disallow use of debugger.
"no-debugger": "error",
// Disallow deletion of variables (deleting properties is fine).
"no-delete-var": "error",
// Allow division operators explicitly at beginning of regular expression.
"no-div-regex": "off",
// Disallow duplicate arguments in functions.
"no-dupe-args": "error",
// Disallow duplicate keys when creating object literals.
"no-dupe-keys": "error",
// Disallow a duplicate case label.
"no-duplicate-case": "error",
// Disallow else after a return in an if. The else around the second return
// here is useless:
// if (something) { return false; } else { return true; }
"no-else-return": "error",
// Disallow empty statements. This will report an error for:
// try { something(); } catch (e) {}
// but will not report it for:
// try { something(); } catch (e) { /* Silencing the error because ...*/ }
// which is a valid use case.
"no-empty": "error",
// Disallow the use of empty character classes in regular expressions.
"no-empty-character-class": "error",
// Disallow use of eval(). We have other APIs to evaluate code in content.
"no-eval": "error",
// Disallow assigning to the exception in a catch block.
"no-ex-assign": "error",
// Disallow adding to native types
"no-extend-native": "error",
// Disallow unnecessary function binding.
"no-extra-bind": "error",
// Disallow double-negation boolean casts in a boolean context.
"no-extra-boolean-cast": "error",
// Allow unnecessary parentheses, as they may make the code more readable.
"no-extra-parens": "off",
// Disallow unnecessary semicolons.
"no-extra-semi": "error",
// Deprecated, will be removed in 1.0.
"no-extra-strict": "off",
// Disallow fallthrough of case statements, except if there is a comment.
"no-fallthrough": "error",
// Allow the use of leading or trailing decimal points in numeric literals.
"no-floating-decimal": "off",
// disallow use of eval()-like methods
"no-implied-eval": "error",
// Allow comments inline after code.
"no-inline-comments": "off",
// Disallow if as the only statement in an else block.
"no-lonely-if": "error",
// Allow mixing regular variable and require declarations (not a node env).
"no-mixed-requires": "off",
// Disallow mixed spaces and tabs for indentation.
"no-mixed-spaces-and-tabs": "error",
// Disallow use of multiple spaces (sometimes used to align const values,
// array or object items, etc.). It's hard to maintain and doesn't add that
// much benefit.
"no-multi-spaces": "error",
// Disallow use of multiline strings (use template strings instead).
"no-multi-str": "error",
// Disallow multiple empty lines.
"no-multiple-empty-lines": [
"error",
{
"max": 1
}
],
// Disallow reassignments of native objects.
"no-native-reassign": "error",
// Disallow nested ternary expressions, they make the code hard to read.
"no-nested-ternary": "error",
// Allow use of new operator with the require function.
"no-new-require": "off",
// Disallow use of octal literals.
"no-octal": "error",
// Allow reassignment of function parameters.
"no-param-reassign": "off",
// Allow string concatenation with __dirname and __filename (not a node env).
"no-path-concat": "off",
// Allow use of unary operators, ++ and --.
"no-plusplus": "off",
// Allow using process.env (not a node environment).
"no-process-env": "off",
// Allow using process.exit (not a node environment).
"no-process-exit": "off",
// Disallow usage of __proto__ property.
"no-proto": "error",
// Disallow declaring the same variable more than once (we use let anyway).
"no-redeclare": "error",
// Disallow multiple spaces in a regular expression literal.
"no-regex-spaces": "off",
// Allow reserved words being used as object literal keys.
"no-reserved-keys": "off",
// Don't restrict usage of specified node modules (not a node environment).
"no-restricted-modules": "off",
// Disallow use of assignment in return statement. It is preferable for a
// single line of code to have only one easily predictable effect.
"no-return-assign": "error",
// Allow use of javascript: urls.
"no-script-url": "off",
// Disallow assignments like foo = foo
"no-self-assign": "error",
// Disallow comparisons where both sides are exactly the same.
"no-self-compare": "error",
// Disallow use of comma operator.
"no-sequences": "error",
// Warn about declaration of variables already declared in the outer scope.
// This isn't an error because it sometimes is useful to use the same name
// in a small helper function rather than having to come up with another
// random name.
// Still, making this a warning can help people avoid being confused.
"no-shadow": "error",
// Disallow shadowing of names such as arguments.
"no-shadow-restricted-names": "error",
// Deprecated, will be removed in 1.0.
"no-space-before-semi": "off",
// Disallow space between function identifier and application.
"no-spaced-func": "error",
// Disallow sparse arrays, eg. let arr = [,,2].
// Array destructuring is fine though:
// for (let [, breakpointPromise] of aPromises)
"no-sparse-arrays": "error",
// Allow use of synchronous methods (not a node environment).
"no-sync": "off",
// Allow the use of ternary operators.
"no-ternary": "off",
// Disallow throwing literals (eg. throw "error" instead of
// throw new Error("error")).
"no-throw-literal": "error",
// Disallow trailing whitespace at the end of lines.
"no-trailing-spaces": "error",
// Disallow use of undeclared variables unless mentioned in a /*global */
// block. Note that globals from head.js are automatically imported in tests
// by the import-headjs-globals rule form the mozilla eslint plugin.
"no-undef": "error",
// Allow dangling underscores in identifiers (for privates).
"no-underscore-dangle": "off",
// Allow use of undefined variable.
"no-undefined": "off",
// Disallow the use of Boolean literals in conditional expressions.
"no-unneeded-ternary": "error",
// Disallow unreachable statements after a return, throw, continue, or break
// statement.
"no-unreachable": "error",
// Disallow global and local variables that aren't used, but allow unused
// function arguments.
"no-unused-vars": [
"error",
{
"vars": "all",
"args": "none"
}
],
// Disallow flow control that escapes from "finally".
"no-unsafe-finally": "error",
// Allow using variables before they are defined.
"no-use-before-define": "off",
// Disallow useless Function.prototype.{call/apply}
"no-useless-call": "error",
// Disallow useless return;
"no-useless-return": "error",
// We use var-only-at-top-level instead of no-var as we allow top level
// vars.
"no-var": "off",
// Allow using TODO/FIXME comments.
"no-warning-comments": "off",
// Disallow use of the with statement.
"no-with": "error",
// Don't require method and property shorthand syntax for object literals.
// We use this in the code a lot, but not consistently, and this seems more
// like something to check at code review time.
"object-shorthand": "off",
// Allow more than one variable declaration per function.
"one-var": "off",
// Disallow padding within blocks.
"padded-blocks": [
"error",
"never"
],
// Don't require quotes around object literal property names.
"quote-props": "off",
// Double quotes should be used. Other quote characters can be used around strings
// with embedded double quotes to avoid escaping them. Template literals are allowed
// mainly for building multi-line messages where only some lines use substitution.
"quotes": [
"error",
"double",
{
"avoidEscape": true,
"allowTemplateLiterals": true
}
],
// Require use of the second argument for parseInt().
"radix": "error",
// Always require use of semicolons wherever they are valid.
"semi": [
"error",
"always"
],
// Enforce spacing after semicolons.
"semi-spacing": [
"error",
{
"before": false,
"after": true
}
],
// Don't require to sort variables within the same declaration block.
// Anyway, one-var is disabled.
"sort-vars": "off",
// Deprecated, will be removed in 1.0.
"space-after-function-name": "off",
// Require a space around all keywords.
"keyword-spacing": "error",
// Require a space before the start brace of a block.
"space-before-blocks": [
"error",
"always"
],
// Deprecated, will be removed in 1.0.
"space-before-function-parentheses": "off",
// Require space after keyword for anonymous functions, but disallow space
// after name of named functions.
"space-before-function-paren": [
"error",
{
"anonymous": "always",
"named": "never"
}
],
// Disable the rule that checks if spaces inside {} and [] are there or not.
// Our code is split on conventions, and it'd be nice to have "error" rules
// instead, one for [] and one for {}. So, disabling until we write them.
"space-in-brackets": "off",
// Disallow spaces inside parentheses.
"space-in-parens": [
"error",
"never"
],
// Require spaces around operators, except for a|"off".
"space-infix-ops": [
"error",
{
"int32Hint": true
}
],
// Require spaces before/after unary operators (words on by default,
// nonwords off by default).
"space-unary-ops": [
"error",
{
"words": true,
"nonwords": false
}
],
// Deprecated, will be removed in 1.0.
"space-unary-word-ops": "off",
// Require a space immediately following the // in a line comment.
"spaced-comment": [
"error",
"always"
],
// Require "use strict" to be defined globally in the script.
"strict": [
"error",
"global"
],
// Disallow comparisons with the value NaN.
"use-isnan": "error",
// Warn about invalid JSDoc comments.
// Disabled for now because of https://github.com/eslint/eslint/issues/2270
// The rule fails on some jsdoc comments like in:
// devtools/client/webconsole/console-output.js
"valid-jsdoc": "off",
// Ensure that the results of typeof are compared against a valid string.
"valid-typeof": "error",
// Allow vars to be declared anywhere in the scope.
"vars-on-top": "off",
// Don't require immediate function invocation to be wrapped in parentheses.
"wrap-iife": "off",
// Don't require regex literals to be wrapped in parentheses (which
// supposedly prevent them from being mistaken for division operators).
"wrap-regex": "off",
// Disallow Yoda conditions (where literal value comes first).
"yoda": "error",
// And these are the rules that haven't been discussed so far, and that are
// disabled for now until we introduce them, one at a time.
// enforce consistent spacing before and after the arrow in arrow functions
"arrow-spacing": "off",
// enforce consistent spacing inside computed property brackets
"computed-property-spacing": "off",
// Require for-in loops to have an if statement.
"guard-for-in": "off",
// allow/disallow an empty newline after var statement
"newline-after-var": "off",
// disallow the use of alert, confirm, and prompt
"no-alert": "off",
// disallow comparisons to null without a type-checking operator
"no-eq-null": "off",
// disallow overwriting functions written as function declarations
"no-func-assign": "off",
// disallow function or variable declarations in nested blocks
"no-inner-declarations": "off",
// disallow invalid regular expression strings in the RegExp constructor
"no-invalid-regexp": "off",
// disallow irregular whitespace outside of strings and comments
"no-irregular-whitespace": "off",
// disallow usage of __iterator__ property
"no-iterator": "off",
// disallow labels that share a name with a variable
"no-label-var": "off",
// disallow use of labeled statements
"no-labels": "error",
// disallow unnecessary nested blocks
"no-lone-blocks": "off",
// disallow creation of functions within loops
"no-loop-func": "off",
// disallow negation of the left operand of an in expression
"no-negated-in-lhs": "off",
// disallow use of new operator when not part of the assignment or
// comparison
"no-new": "off",
// disallow use of new operator for Function object
"no-new-func": "off",
// disallow use of the Object constructor
"no-new-object": "off",
// disallows creating new instances of String,Number, and Boolean
"no-new-wrappers": "off",
// disallow the use of object properties of the global object (Math and
// JSON) as functions
"no-obj-calls": "off",
// disallow use of octal escape sequences in string literals, such as
// var foo = "Copyright \251";
"no-octal-escape": "off",
// disallow use of undefined when initializing variables
"no-undef-init": "off",
// disallow usage of expressions in statement position
"no-unused-expressions": "off",
// disallow unnecessary concatenation of literals or template literals
"no-useless-concat": "off",
// disallow use of void operator
"no-void": "off",
// disallow wrapping of non-IIFE statements in parens
"no-wrap-func": "off",
// require assignment operator shorthand where possible or prohibit it
// entirely
"operator-assignment": "off",
// enforce operators to be placed before or after line breaks
"operator-linebreak": "off"
}
}