Merge pull request #47 from blond/issue-34

Add `class-methods-use-this` rule
blond · Apr 22, 2017 · 9ae4a9b · 9ae4a9b
2 parents 926a611 + df6c7ed
commit 9ae4a9b
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 1 deletion.
diff --git a/examples/bad/es6.js b/examples/bad/es6.js
@@ -37,3 +37,14 @@ var foo = new Symbol('foo'); // The Symbol constructor is not intended to be use
 function* foo() {
     return 10; // This generator functions that do not have the `yield` keyword.
 }
+
+/*eslint class-methods-use-this: "error"*/
+class A {
+    constructor() {
+        this.a = "hi";
+    }
+
+    sayHi() {
+        console.log("hi"); // The sayHi method doesn’t use this, so we can make it a static method.
+    }
+}
diff --git a/lib/es6-rules.js b/lib/es6-rules.js
@@ -10,5 +10,6 @@ module.exports = {
     'no-class-assign': 'error',
     'no-const-assign': 'error',
     'no-new-symbol': 'error',
-    'require-yield': 'error'
+    'require-yield': 'error',
+    'class-methods-use-this': 'error'
 };