Issue 3692 - Created global eslintrc

eslintrc.json

Index: eslintrc.json
===================================================================
new file mode 100644
--- /dev/null
+++ b/eslintrc.json
@@ -0,0 +1,56 @@
+{
+  "extends": "eslint:recommended",
+  "rules": {
+
+    /* MDN General practices */
+    "no-else-return": "error",
+    "no-alert": "error",
+    "indent": ["error", 2],
+    "valid-jsdoc": "error",
+    "eol-last": "error",
+    "vars-on-top": "off",

[Sebastian Noack, 2017/02/01 19:50:34]

According to the documentation, it seems vars-on-top isn't included in
eslint:recommended. So might overriding this option be redundant?

+
+    /* MDN Naming and formatting code */
+    "no-trailing-spaces": "error",
+    "max-len": ["error", 80],
+    "brace-style": ["error", "allman"],
+
+    /* MDN JavaScript practices */
+    "consistent-return": "error",
+    "no-array-constructor": "error",
+    "no-new-object": "error",
+
+    /* ABP General */
+    "no-warning-comments": "error",
+
+    /* Palant recommends */
+    "no-unused-vars": "off",

[Sebastian Noack, 2017/02/01 19:50:34]

I just noticed, instead of completely disabling no-unused-vars, we could rather
configure it to ignore global variables and function arguments, so that it still
catches unused local variables, which I think have no legit use whatsoever:

  "no-unused-vars": ["error", {"vars": "local", "args": "none"}]

+    "array-bracket-spacing": "error",
+    "arrow-spacing": "error",
+    "comma-spacing": "error",
+    "comma-style": "error",
+    "computed-property-spacing": "error",
+    "key-spacing": "error",
+    "keyword-spacing": "error",
+    "new-parens": "error",
+    "func-call-spacing": "error",
+    "no-var": "error",
+    "no-whitespace-before-property": "error",
+    "object-curly-spacing": "error",
+    "operator-assignment": "error",
+    "operator-linebreak": "error",
+    "padded-blocks": ["error", "never"],
+    "prefer-rest-params": "error",
+    "prefer-spread": "error",
+    "require-yield": "error",

[Sebastian Noack, 2017/02/01 19:50:34]

It seems this is already included in eslint:recommended. So might overriding
this option be redundant?

+    "quotes": ["error", "double", {"avoidEscape": true}],
+    "semi": "error",
+    "semi-spacing": "error",
+    "space-before-function-paren": ["error", "never"],
+    "space-in-parens": "error",
+    "space-infix-ops": "error",
+    "space-unary-ops": "error",
+    "spaced-comment": "error",
+    "yield-star-spacing": "error"
+  }
+}