feat: update error handling for yaml inputs (#1395)

Co-authored-by: GitHub Action <action@github.com>
tj-actions · Jul 20, 2023 · de0eba3 · de0eba3
1 parent bc262c2
commit de0eba3
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 14 deletions.
diff --git a/dist/index.js b/dist/index.js
diff --git a/dist/index.js.map b/dist/index.js.map
diff --git a/src/utils.ts b/src/utils.ts
@@ -893,17 +893,17 @@ const getYamlFilePatternsFromContents = async ({
 
   if (doc.errors.length > 0) {
     if (filePath) {
-      core.warning(`YAML errors in ${filePath}: ${doc.errors}`)
+      throw new Error(`YAML errors in ${filePath}: ${doc.errors}`)
     } else {
-      core.warning(`YAML errors: ${doc.errors}`)
+      throw new Error(`YAML errors: ${doc.errors}`)
     }
   }
 
   if (doc.warnings.length > 0) {
     if (filePath) {
-      core.warning(`YAML warnings in ${filePath}: ${doc.warnings}`)
+      throw new Error(`YAML warnings in ${filePath}: ${doc.warnings}`)
     } else {
-      core.warning(`YAML warnings: ${doc.warnings}`)
+      throw new Error(`YAML warnings: ${doc.warnings}`)
     }
   }
 
@@ -1164,9 +1164,7 @@ export const hasLocalGitDirectory = async ({
 }: {
   workingDirectory: string
 }): Promise<boolean> => {
-  const insideWorkTree = await isInsideWorkTree({
+  return await isInsideWorkTree({
     cwd: workingDirectory
   })
-
-  return insideWorkTree
 }