Simplify no-console implementation

protofire · Feb 8, 2023 · 7110266 · 7110266
1 parent 0ced110
commit 7110266
Showing 1 changed file with 5 additions and 6 deletions.
diff --git a/lib/rules/best-practises/no-console.js b/lib/rules/best-practises/no-console.js
@@ -39,7 +39,7 @@ class NoConsoleLogChecker extends BaseChecker {
   }
 
   ImportDirective(node) {
-    if (this.isAnImport(node)) {
+    if (this.isConsoleImport(node)) {
       this.error(node, 'Unexpected import of console file')
     }
   }
@@ -59,12 +59,11 @@ class NoConsoleLogChecker extends BaseChecker {
     )
   }
 
-  isAnImport(node) {
+  isConsoleImport(node) {
     return (
-      node.type === 'ImportDirective' &&
-      (node.path === 'hardhat/console.sol' ||
-        node.path === 'forge-std/console.sol' ||
-        node.path === 'forge-std/console2.sol')
+      node.path === 'hardhat/console.sol' ||
+      node.path === 'forge-std/console.sol' ||
+      node.path === 'forge-std/console2.sol'
     )
   }
 }