Ignore whitespace only lines too

remarkjs · May 26, 2020 · dc4f092 · dc4f092
1 parent 3c896b3
commit dc4f092
Showing 1 changed file with 5 additions and 3 deletions.
diff --git a/packages/remark-lint-no-shell-dollars/index.js b/packages/remark-lint-no-shell-dollars/index.js
@@ -90,9 +90,7 @@ function noShellDollars(tree, file) {
 
     // Check both known shell code and unknown code.
     if (!generated(node) && node.lang && flags.indexOf(node.lang) !== -1) {
-      lines = node.value.split('\n').filter(function (line) {
-        return line
-      })
+      lines = node.value.split('\n').filter(notEmpty)
       length = lines.length
       index = -1
 
@@ -111,4 +109,8 @@ function noShellDollars(tree, file) {
       file.message(reason, node)
     }
   }
+
+  function notEmpty(line) {
+    return line.trim().length !== 0
+  }
 }