From 143607bc2ee728f5cd33939d6af2d6dd854669f5 Mon Sep 17 00:00:00 2001 From: Lucas Gravley <29484535+admiralAwkbar@users.noreply.github.com> Date: Tue, 7 Jan 2020 14:38:05 -0600 Subject: [PATCH 1/2] Adding logic to skip ansible on read only --- lib/linter.sh | 55 ++++++++++++++++++++++++++++++++++++++++++++++++--- 1 file changed, 52 insertions(+), 3 deletions(-) diff --git a/lib/linter.sh b/lib/linter.sh index 816409fb..05796375 100755 --- a/lib/linter.sh +++ b/lib/linter.sh @@ -87,6 +87,7 @@ ERRORS_FOUND_PYTHON=0 # Count of errors found ERRORS_FOUND_COFFEE=0 # Count of errors found ERRORS_FOUND_ANSIBLE=0 # Count of errors found ERRORS_FOUND_JAVASCRIPT=0 # Count of errors found +READ_ONLY_CHANGE_FLAG=0 # Flag set to 1 if files changed are not txt or md ################################################################################ ########################## FUNCTIONS BELOW ##################################### @@ -1757,6 +1758,14 @@ LintAnsibleFiles() LIST_FILES=($(cd "$ANSIBLE_DIRECTORY"; ls | grep ".yml" 2>&1)) fi + ############################################################### + # Set the list to empty if only MD and TXT files were changed # + ############################################################### + # No need to run the full ansible checks on read only file changes + if [ "$READ_ONLY_CHANGE_FLAG" eq 0 ]; then + LIST_FILES=() + fi + ################## # Lint the files # ################## @@ -2211,6 +2220,10 @@ BuildFileList() # Append the file to the array # ################################ FILE_ARRAY_YML+=("$FILE") + ########################################################## + # Set the READ_ONLY_CHANGE_FLAG since this could be exec # + ########################################################## + READ_ONLY_CHANGE_FLAG=1 ###################### # Get the JSON files # ###################### @@ -2219,6 +2232,10 @@ BuildFileList() # Append the file to the array # ################################ FILE_ARRAY_JSON+=("$FILE") + ########################################################## + # Set the READ_ONLY_CHANGE_FLAG since this could be exec # + ########################################################## + READ_ONLY_CHANGE_FLAG=1 ##################### # Get the XML files # ##################### @@ -2227,6 +2244,10 @@ BuildFileList() # Append the file to the array # ################################ FILE_ARRAY_XML+=("$FILE") + ########################################################## + # Set the READ_ONLY_CHANGE_FLAG since this could be exec # + ########################################################## + READ_ONLY_CHANGE_FLAG=1 ########################## # Get the MARKDOWN files # ########################## @@ -2243,6 +2264,10 @@ BuildFileList() # Append the file to the array # ################################ FILE_ARRAY_BASH+=("$FILE") + ########################################################## + # Set the READ_ONLY_CHANGE_FLAG since this could be exec # + ########################################################## + READ_ONLY_CHANGE_FLAG=1 ###################### # Get the PERL files # ###################### @@ -2251,6 +2276,10 @@ BuildFileList() # Append the file to the array # ################################ FILE_ARRAY_PERL+=("$FILE") + ########################################################## + # Set the READ_ONLY_CHANGE_FLAG since this could be exec # + ########################################################## + READ_ONLY_CHANGE_FLAG=1 ###################### # Get the RUBY files # ###################### @@ -2259,6 +2288,10 @@ BuildFileList() # Append the file to the array # ################################ FILE_ARRAY_RUBY+=("$FILE") + ########################################################## + # Set the READ_ONLY_CHANGE_FLAG since this could be exec # + ########################################################## + READ_ONLY_CHANGE_FLAG=1 ######################## # Get the PYTHON files # ######################## @@ -2267,6 +2300,10 @@ BuildFileList() # Append the file to the array # ################################ FILE_ARRAY_PYTHON+=("$FILE") + ########################################################## + # Set the READ_ONLY_CHANGE_FLAG since this could be exec # + ########################################################## + READ_ONLY_CHANGE_FLAG=1 ######################## # Get the COFFEE files # ######################## @@ -2275,19 +2312,31 @@ BuildFileList() # Append the file to the array # ################################ FILE_ARRAY_COFFEE+=("$FILE") - ######################## - # Get the COFFEE files # - ######################## + ########################################################## + # Set the READ_ONLY_CHANGE_FLAG since this could be exec # + ########################################################## + READ_ONLY_CHANGE_FLAG=1 + ############################ + # Get the JavaScript files # + ############################ elif [ "$FILE_TYPE" == "js" ]; then ################################ # Append the file to the array # ################################ FILE_ARRAY_JAVASCRIPT+=("$FILE") + ########################################################## + # Set the READ_ONLY_CHANGE_FLAG since this could be exec # + ########################################################## + READ_ONLY_CHANGE_FLAG=1 else ############################ # Extension was not found! # ############################ echo " - WARN! Failed to get filetype for:[$FILE]!" + ########################################################## + # Set the READ_ONLY_CHANGE_FLAG since this could be exec # + ########################################################## + READ_ONLY_CHANGE_FLAG=1 fi done From c507cf594374787696f599dd55d449c2907b52dd Mon Sep 17 00:00:00 2001 From: Lucas Gravley <29484535+admiralAwkbar@users.noreply.github.com> Date: Tue, 7 Jan 2020 14:41:12 -0600 Subject: [PATCH 2/2] fix mistake --- lib/linter.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/linter.sh b/lib/linter.sh index 05796375..5f0174bd 100755 --- a/lib/linter.sh +++ b/lib/linter.sh @@ -1762,7 +1762,7 @@ LintAnsibleFiles() # Set the list to empty if only MD and TXT files were changed # ############################################################### # No need to run the full ansible checks on read only file changes - if [ "$READ_ONLY_CHANGE_FLAG" eq 0 ]; then + if [ "$READ_ONLY_CHANGE_FLAG" -eq 0 ]; then LIST_FILES=() fi