Bug 1246904 - Change devtools eslint warnings to be errors instead; r=jryans

MozReview-Commit-ID: BBE2kZKF1cY

devtools/.eslintrc

@@ -35,18 +35,18 @@
     "block-scoped-var": 2,
     // Enforce one true brace style (opening brace on the same line) and avoid
     // start and end braces on the same line.
-    "brace-style": [1, "1tbs", {"allowSingleLine": false}],
+    "brace-style": [2, "1tbs", {"allowSingleLine": false}],
     // Require camel case names
-    "camelcase": 1,
+    "camelcase": 2,
     // Allow trailing commas for easy list extension.  Having them does not
     // impair readability, but also not required either.
     "comma-dangle": 0,
     // Enforce spacing before and after comma
-    "comma-spacing": [1, {"before": false, "after": true}],
+    "comma-spacing": [2, {"before": false, "after": true}],
     // Enforce one true comma style.
-    "comma-style": [1, "last"],
+    "comma-style": [2, "last"],
     // Warn about cyclomatic complexity in functions.
-    "complexity": 1,
+    "complexity": 2,
     // Require return statements to either always or never specify values.
     "consistent-return": 2,
     // Don't warn for inconsistent naming when capturing this (not so important
@@ -58,7 +58,7 @@
     // add a bogus default when you know all possible cases are handled.
     "default-case": 0,
     // Enforce dots on the next line with property name.
-    "dot-location": [1, "property"],
+    "dot-location": [2, "property"],
     // Encourage the use of dot notation whenever possible.
     "dot-notation": 2,
     // Enforce newline at the end of file, with no multiple empty lines.
@@ -77,15 +77,15 @@
     // Deprecated, will be removed in 1.0.
     "generator-star": 0,
     // Enforce the spacing around the * in generator functions.
-    "generator-star-spacing": [1, "after"],
+    "generator-star-spacing": [2, "after"],
     // Deprecated, will be removed in 1.0.
     "global-strict": 0,
     // Only useful in a node environment.
     "handle-callback-err": 0,
     // Tab width.
-    "indent": [1, 2, {"SwitchCase": 1}],
+    "indent": [2, 2, {"SwitchCase": 1}],
     // Enforces spacing between keys and values in object literal properties.
-    "key-spacing": [1, {"beforeColon": false, "afterColon": true}],
+    "key-spacing": [2, {"beforeColon": false, "afterColon": true}],
     // Allow mixed 'LF' and 'CRLF' as linebreaks.
     "linebreak-style": 0,
     // Don't enforce the maximum depth that blocks can be nested. The complexity
@@ -115,7 +115,7 @@
     "no-caller": 2,
     // Disallow the catch clause parameter name being the same as a variable in
     // the outer scope, to avoid confusion.
-    "no-catch-shadow": 1,
+    "no-catch-shadow": 2,
     // Deprecated, will be removed in 1.0.
     "no-comma-dangle": 0,
     // Disallow assignment in conditional expressions.
@@ -167,7 +167,7 @@
     // Allow unnecessary parentheses, as they may make the code more readable.
     "no-extra-parens": 0,
     // Disallow unnecessary semicolons.
-    "no-extra-semi": 1,
+    "no-extra-semi": 2,
     // Deprecated, will be removed in 1.0.
     "no-extra-strict": 0,
     // Disallow fallthrough of case statements, except if there is a comment.
@@ -175,7 +175,7 @@
     // Allow the use of leading or trailing decimal points in numeric literals.
     "no-floating-decimal": 0,
     // Disallow comments inline after code.
-    "no-inline-comments": 1,
+    "no-inline-comments": 2,
     // Disallow if as the only statement in an else block.
     "no-lonely-if": 2,
     // Allow mixing regular variable and require declarations (not a node env).
@@ -185,11 +185,11 @@
     // 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": 1,
+    "no-multi-spaces": 2,
     // Disallow use of multiline strings (use template strings instead).
-    "no-multi-str": 1,
+    "no-multi-str": 2,
     // Disallow multiple empty lines.
-    "no-multiple-empty-lines": [1, {"max": 1}],
+    "no-multiple-empty-lines": [2, {"max": 1}],
     // Disallow reassignments of native objects.
     "no-native-reassign": 2,
     // Disallow nested ternary expressions, they make the code hard to read.
@@ -197,7 +197,7 @@
     // Allow use of new operator with the require function.
     "no-new-require": 0,
     // Disallow use of octal literals.
-    "no-octal": 1,
+    "no-octal": 2,
     // Allow reassignment of function parameters.
     "no-param-reassign": 0,
     // Allow string concatenation with __dirname and __filename (not a node env).
@@ -232,13 +232,13 @@
     // 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": 1,
+    "no-shadow": 2,
     // Disallow shadowing of names such as arguments.
     "no-shadow-restricted-names": 2,
     // Deprecated, will be removed in 1.0.
     "no-space-before-semi": 0,
     // Disallow space between function identifier and application.
-    "no-spaced-func": 1,
+    "no-spaced-func": 2,
     // Disallow sparse arrays, eg. let arr = [,,2].
     // Array destructuring is fine though:
     // for (let [, breakpointPromise] of aPromises)
@@ -283,47 +283,47 @@
     // Allow more than one variable declaration per function.
     "one-var": 0,
     // Disallow padding within blocks.
-    "padded-blocks": [1, "never"],
+    "padded-blocks": [2, "never"],
     // Don't require quotes around object literal property names.
     "quote-props": 0,
     // Double quotes should be used.
-    "quotes": [1, "double", "avoid-escape"],
+    "quotes": [2, "double", "avoid-escape"],
     // Require use of the second argument for parseInt().
     "radix": 2,
     // Always require use of semicolons wherever they are valid.
-    "semi": [1, "always"],
+    "semi": [2, "always"],
     // Enforce spacing after semicolons.
-    "semi-spacing": [1, {"before": false, "after": true}],
+    "semi-spacing": [2, {"before": false, "after": true}],
     // Don't require to sort variables within the same declaration block.
     // Anyway, one-var is disabled.
     "sort-vars": 0,
     // Deprecated, will be removed in 1.0.
     "space-after-function-name": 0,
     // Require a space after keywords.
-    "space-after-keywords": [1, "always"],
+    "space-after-keywords": [2, "always"],
     // Require a space before the start brace of a block.
-    "space-before-blocks": [1, "always"],
+    "space-before-blocks": [2, "always"],
     // Deprecated, will be removed in 1.0.
     "space-before-function-parentheses": 0,
     // Disallow space before function opening parenthesis.
-    "space-before-function-paren": [1, "never"],
+    "space-before-function-paren": [2, "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 2 rules
     // instead, one for [] and one for {}. So, disabling until we write them.
     "space-in-brackets": 0,
     // Disallow spaces inside parentheses.
-    "space-in-parens": [1, "never"],
+    "space-in-parens": [2, "never"],
     // Require spaces around operators, except for a|0.
-    "space-infix-ops": [1, {"int32Hint": true}],
+    "space-infix-ops": [2, {"int32Hint": true}],
     // Require a space after return, throw, and case.
-    "space-return-throw-case": 1,
+    "space-return-throw-case": 2,
     // Require spaces before/after unary operators (words on by default,
     // nonwords off by default).
-    "space-unary-ops": [1, { "words": true, "nonwords": false }],
+    "space-unary-ops": [2, { "words": true, "nonwords": false }],
     // Deprecated, will be removed in 1.0.
     "space-unary-word-ops": 0,
     // Require a space immediately following the // in a line comment.
-    "spaced-comment": [1, "always"],
+    "spaced-comment": [2, "always"],
     // Require "use strict" to be defined globally in the script.
     "strict": [2, "global"],
     // Disallow comparisons with the value NaN.