Bug 1249627 - Eslint: allow unused arguments; r=pbrosset
authorJan Odvarko <odvarko@gmail.com>
Fri, 19 Feb 2016 16:30:29 +0100
changeset 321476 b795629bce7f88105dadada175a3cdfb2198543f
parent 321475 4d32d1f87b74a73f07ce343dd3e48c7c5c1eb099
child 321477 cad377e8dffcf582fd4211c4215659f3aeb29c3d
push id5913
push userjlund@mozilla.com
push dateMon, 25 Apr 2016 16:57:49 +0000
treeherdermozilla-beta@dcaf0a6fa115 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewerspbrosset
bugs1249627
milestone47.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1249627 - Eslint: allow unused arguments; r=pbrosset MozReview-Commit-ID: AjNppE9zLWL
devtools/.eslintrc
--- a/devtools/.eslintrc
+++ b/devtools/.eslintrc
@@ -260,18 +260,18 @@
     "no-underscore-dangle": 0,
     // Allow use of undefined variable.
     "no-undefined": 0,
     // Disallow the use of Boolean literals in conditional expressions.
     "no-unneeded-ternary": 2,
     // Disallow unreachable statements after a return, throw, continue, or break
     // statement.
     "no-unreachable": 2,
-    // Disallow declaration of variables that are not used in the code
-    "no-unused-vars": 2,
+    // Disallow global and local variables that aren't used, but allow unused function arguments.
+    "no-unused-vars": [2, {"vars": "all", "args": "none"}],
     // Allow using variables before they are defined.
     "no-use-before-define": 0,
     // We use var-only-at-top-level instead of no-var as we allow top level
     // vars.
     "no-var": 0,
     // Allow using TODO/FIXME comments.
     "no-warning-comments": 0,
     // Disallow use of the with statement.