Given that this patch causes a lot of churn in the addon code, I wouldn't really mind if we ultimately decide against doing this and just add a rule exception in mozilla-central instead.[1] --- [1] Note that I used the ESLint `--fix` option, hence writing this commit message actually took longer time than the creation of the patch :-)
35 lines
842 B
Plaintext
35 lines
842 B
Plaintext
{
|
|
"extends": [
|
|
../../.eslintrc
|
|
],
|
|
|
|
"parserOptions": {
|
|
"ecmaVersion": 6
|
|
},
|
|
|
|
"rules": {
|
|
// Variables
|
|
"no-shadow": "error",
|
|
"no-unused-vars": ["error", {
|
|
"vars": "local",
|
|
"varsIgnorePattern": "^Cc|Ci|Cu|Cr|EXPORTED_SYMBOLS",
|
|
"args": "none",
|
|
}],
|
|
|
|
// Stylistic Issues
|
|
"quotes": ["error", "double"],
|
|
"space-before-function-paren": ["error", "never"],
|
|
|
|
// ECMAScript 6
|
|
"arrow-body-style": ["error", "as-needed"],
|
|
"arrow-parens": ["error", "always"],
|
|
"arrow-spacing": ["error", { "before": true, "after": true, }],
|
|
"constructor-super": "error",
|
|
"no-confusing-arrow": "error",
|
|
"no-const-assign": "error",
|
|
"no-dupe-class-members": "error",
|
|
"no-useless-constructor": "error",
|
|
"object-shorthand": ["error", "always", { "avoidQuotes": true }],
|
|
},
|
|
}
|