Do not use class-methods-use-this

Author: robertrossmann

standard/best-practices.js

@@ -14,10 +14,6 @@ module.exports = {
     // This rule enforces usage of return statement in callbacks of array's methods.
     'array-callback-return': 1,
 
-    // Enforce that class methods utilize this
-    // If a class method does not use `this`, it can safely be made a static function.
-    'class-methods-use-this': 1,
-
     // This rule enforces consistent use of trailing commas in object and array literals
     // Allow trailing commas for func parameters, array and object literals spread across
     // multiple lines

unused.js

@@ -14,6 +14,11 @@ module.exports = {
     // Unused in favour of `no-var` rule. Front-end could have some interest in this rule.
     'block-scoped-var': 0,
 
+    // Enforce that class methods utilize `this`
+    // Unused, many a time you need to add a method on a class just to make it adhere to a
+    // prescribed API (ie. React components).
+    'class-methods-use-this': 0,
+
     // Blacklist certain identifiers
     // Unused, too restrictive.
     'id-blacklist': 0,