Bail out early if no relevant files

In a run where there are no relevant files, ESLint v6 was failing because the list passed in was empty. It was rendering the help listing instead of returning JSON that could be parsed.
planningcenter · Oct 8, 2024 · 70f8eaa · 70f8eaa
1 parent 59a5f16
commit 70f8eaa
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 0 deletions.
diff --git a/dist/index.js b/dist/index.js
diff --git a/src/index.ts b/src/index.ts
@@ -43,6 +43,10 @@ async function run() {
     `Changed files matching extensions: ${changedFilesMatchingExtensions}`,
   )
 
+  // Bail out early if the file list is empty (older ESLint versions will
+  // complain if the list is empty)
+  if (changedFilesMatchingExtensions.length === 0) return
+
   let { stdout: eslintOut, exitCode } = await getExecOutput(
     "npx eslint --format=json",
     changedFilesMatchingExtensions,