From 116698b03237fed27fa86ab9e4ebc1e4ac1266e0 Mon Sep 17 00:00:00 2001
From: Lucas Gravley <29484535+admiralAwkbar@users.noreply.github.com>
Date: Fri, 26 Jun 2020 08:26:36 -0500
Subject: [PATCH] fixed linting errors

---
 lib/linter.sh | 33 ++++++++++++++++++++++++++++-----
 1 file changed, 28 insertions(+), 5 deletions(-)

diff --git a/lib/linter.sh b/lib/linter.sh
index a89bb3fe..06bc0022 100755
--- a/lib/linter.sh
+++ b/lib/linter.sh
@@ -573,10 +573,33 @@ LintAnsibleFiles()
 #### Function DetectOpenAPIFile ################################################
 DetectOpenAPIFile()
 {
-  egrep '"openapi":|"swagger":|^openapi:|^swagger:' $GITHUB_WORKSPACE/$1 > /dev/null
-  if [ $? -eq 0 ]; then
+  ################
+  # Pull in vars #
+  ################
+  FILE="$1"
+
+  ###############################
+  # Check the file for keywords #
+  ###############################
+  egrep '"openapi":|"swagger":|^openapi:|^swagger:' "$GITHUB_WORKSPACE/$FILE" > /dev/null
+
+  #######################
+  # Load the error code #
+  #######################
+  ERROR_CODE=$?
+
+  ##############################
+  # Check the shell for errors #
+  ##############################
+  if [ $ERROR_CODE -eq 0 ]; then
+    ########################
+    # Found string in file #
+    ########################
 	  return 0
   else
+    ###################
+    # No string match #
+    ###################
 	  return 1
   fi
 }
@@ -1460,7 +1483,7 @@ BuildFileList()
       ############################
       # Check if file is OpenAPI #
       ############################
-      if DetectOpenAPIFile $FILE; then
+      if DetectOpenAPIFile "$FILE"; then
         FILE_ARRAY_OPENAPI+=("$FILE")
       fi
       ##########################################################
@@ -1478,7 +1501,7 @@ BuildFileList()
       ############################
       # Check if file is OpenAPI #
       ############################
-      if DetectOpenAPIFile $FILE; then
+      if DetectOpenAPIFile "$FILE"; then
         FILE_ARRAY_OPENAPI+=("$FILE")
       fi
       ##########################################################
@@ -2640,7 +2663,7 @@ if [ "$VALIDATE_OPENAPI" == "true" ]; then
     LIST_FILES=($(cd "$GITHUB_WORKSPACE" || exit; find . -type f -regex ".*\.\(yml\|yaml\|json\)\$" 2>&1))
     for FILE in "${LIST_FILES[@]}"
     do
-      if DetectOpenAPIFile $FILE; then
+      if DetectOpenAPIFile "$FILE"; then
         FILE_ARRAY_OPENAPI+=("$FILE")
       fi
     done