Fix: eol-last allow empty-string to always pass (refs #9534) (#9696)

Note that there are already tests asserting this behavior, but they aren't actually run in the rules. See #9534
eslint · Dec 11, 2017 · b72dc83 · b72dc83
1 parent d80aa7c
commit b72dc83
Showing 1 changed file with 8 additions and 0 deletions.
diff --git a/lib/rules/eol-last.js b/lib/rules/eol-last.js
@@ -46,6 +46,14 @@ module.exports = {
                     CRLF = `\r${LF}`,
                     endsWithNewline = lodash.endsWith(src, LF);
 
+                /*
+                 * Empty source is always valid: No content in file so we don't
+                 * need to lint for a newline on the last line of content.
+                 */
+                if (!src.length) {
+                    return;
+                }
+
                 let mode = context.options[0] || "always",
                     appendCRLF = false;