Bug 1551218 - Part 2: Remove devtools debugger eslint rules conflicting with any prettier configuration, r=Standard8,jlast

Differential Revision: https://phabricator.services.mozilla.com/D30927

--HG--
extra : moz-landing-system : lando

devtools/client/debugger/.eslintrc

@@ -83,9 +83,6 @@
     "import/named": "error",
     "import/export": "error",
 
-    // Enforce the spacing around the * in generator functions.
-    "generator-star-spacing": [2, "after"],
-
     "flowtype/define-flow-type": 1,
     "flowtype/use-flow-type": 1,
 
@@ -97,9 +94,6 @@
     "block-scoped-var": 2,
     // Require camel case names
     "camelcase": 2,
-    // Allow trailing commas for easy list extension.  Having them does not
-    // impair readability, but also not required either.
-    "comma-dangle": 0,
     // Warn about cyclomatic complexity in functions.
     "complexity": ["error", { "max": 22 }],
     // Don't warn for inconsistent naming when capturing this (not so important
@@ -124,26 +118,12 @@
     // Allow use of function declarations and expressions.
     "func-style": 0,
     // Deprecated, will be removed in 1.0.
-    "generator-star": 0,
-    // Deprecated, will be removed in 1.0.
     "global-strict": 0,
     // Only useful in a node environment.
     "handle-callback-err": 0,
-    // Allow mixed 'LF' and 'CRLF' as linebreaks.
-    "linebreak-style": 0,
     // Don't enforce the maximum depth that blocks can be nested. The complexity
     // rule is a better rule to check this.
     "max-depth": 0,
-    // Maximum length of a line.
-    "max-len": [
-      2,
-      80,
-      2,
-      {
-        "ignoreUrls": true,
-        "ignorePattern": "\\s*require\\s*\\(|^\\s*loader\\.lazy|-\\*-"
-      }
-    ],
     // Maximum depth callbacks can be nested.
     "max-nested-callbacks": [2, 4],
     // Don't limit the number of parameters that can be used in a function.
@@ -164,8 +144,6 @@
     // Disallow the catch clause parameter name being the same as a variable in
     // the outer scope, to avoid confusion.
     "no-catch-shadow": 2,
-    // Deprecated, will be removed in 1.0.
-    "no-comma-dangle": 0,
     // Disallow assignment in conditional expressions.
     "no-cond-assign": 2,
     // Allow using the console API.
@@ -212,14 +190,10 @@
     "no-extra-bind": 2,
     // Disallow double-negation boolean casts in a boolean context.
     "no-extra-boolean-cast": 2,
-    // Allow unnecessary parentheses, as they may make the code more readable.
-    "no-extra-parens": 0,
     // Deprecated, will be removed in 1.0.
     "no-extra-strict": 0,
     // Disallow fallthrough of case statements, except if there is a comment.
     "no-fallthrough": 2,
-    // Allow the use of leading or trailing decimal points in numeric literals.
-    "no-floating-decimal": 0,
     // Disallow comments inline after code.
     "no-inline-comments": 2,
     // Disallow if as the only statement in an else block.
@@ -260,8 +234,6 @@
     "no-redeclare": 2,
     // Disallow multiple spaces in a regular expression literal.
     "no-regex-spaces": 2,
-    // Allow reserved words being used as object literal keys.
-    "no-reserved-keys": 0,
     // Don't restrict usage of specified node modules (not a node environment).
     "no-restricted-modules": 0,
     // Disallow use of assignment in return statement. It is preferable for a
@@ -281,8 +253,6 @@
     "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 sparse arrays, eg. let arr = [,,2].
     // Array destructuring is fine though:
     // for (let [, breakpointPromise] of aPromises)
@@ -324,29 +294,11 @@
     "object-shorthand": 0,
     // Allow more than one variable declaration per function.
     "one-var": 0,
-    // Disallow padding within blocks.
-    //"padded-blocks": [2, "never"],
-    // Dont require quotes around object literal property names.
-    "quote-props": 0,
-    // Double quotes should be used.
-    "quotes": [2, "double", "avoid-escape"],
     // Require use of the second argument for parseInt().
     "radix": 2,
     // Dont 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,
-    // Deprecated, will be removed in 1.0.
-    "space-before-function-parentheses": 0,
-    // Disallow space before function opening parenthesis.
-    //"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 itd be nice to have 2 rules
-    // instead, one for [] and one for {}. So, disabling until we write them.
-    "space-in-brackets": 0,
-    // Deprecated, will be removed in 1.0.
-    "space-unary-word-ops": 0,
     // Require a space immediately following the // in a line comment.
     "spaced-comment": [2, "always"],
     // Require "use strict" to be defined globally in the script.
@@ -362,11 +314,6 @@
     "valid-typeof": 2,
     // Allow vars to be declared anywhere in the scope.
     "vars-on-top": 0,
-    // Dont require immediate function invocation to be wrapped in parentheses.
-    "wrap-iife": 0,
-    // Don't require regex literals to be wrapped in parentheses (which
-    // supposedly prevent them from being mistaken for division operators).
-    "wrap-regex": 0,
     // Disallow Yoda conditions (where literal value comes first).
     "yoda": 2,
 
@@ -422,13 +369,9 @@
     "no-unused-expressions": 0,
     // disallow use of void operator
     "no-void": 0,
-    // disallow wrapping of non-IIFE statements in parens
-    "no-wrap-func": 0,
     // require assignment operator shorthand where possible or prohibit it
     // entirely
     "operator-assignment": 0,
-    // enforce operators to be placed before or after line breaks
-    "operator-linebreak": 0,
 
     // Rules from the prettier plugin
     "prettier/prettier": "error",