From 4a6efba060bcfb6a9391e492d4021afa8af9dbf7 Mon Sep 17 00:00:00 2001 From: Lucas Gravley <29484535+admiralAwkbar@users.noreply.github.com> Date: Thu, 12 Dec 2019 10:34:58 -0600 Subject: [PATCH 1/2] adding env --- Dockerfile | 2 +- lib/linter.sh | 50 +++++++++++++++++++++++++++++++++++++++++++++++++- 2 files changed, 50 insertions(+), 2 deletions(-) diff --git a/Dockerfile b/Dockerfile index 0962e87e..300acbf9 100644 --- a/Dockerfile +++ b/Dockerfile @@ -32,7 +32,7 @@ RUN apk add --no-cache \ # Run Pip3 Installs # ##################### RUN pip3 --no-cache-dir install --upgrade --no-cache-dir \ - yamllint pylint + yamllint pylint yq #################### # Run NPM Installs # diff --git a/lib/linter.sh b/lib/linter.sh index e8c5995c..842f12bd 100755 --- a/lib/linter.sh +++ b/lib/linter.sh @@ -1545,6 +1545,54 @@ Eslint() #### Function StandardLint ##################################################### StandardLint() { + ######################################################################### + # Need to get the ENV vars from the linter rules to run in command line # + ######################################################################### + # Set the IFS to newline + IFS=$'\n' + + ######################################### + # Get list of all environment variables # + ######################################### + # Only env vars that are marked as true + GET_ENV_ARRAY=($(yq .env "$JAVASCRIPT_LINTER_RULES" |grep true)) + + ####################### + # Load the error code # + ####################### + ERROR_CODE=$? + + ############################## + # Check the shell for errors # + ############################## + if [ $ERROR_CODE -ne 0 ]; then + # ERROR + echo "ERROR! Failed to gain list of ENV vars to load!" + echo "ERROR:[${GET_ENV_ARRAY[*]}]" + exit 1 + fi + + ###################### + # Set the env string # + ###################### + ENV_STRING='' + + ############################# + # Pull out the envs to load # + ############################# + for ENV in "${GET_ENV_ARRAY[@]}" + do + ############################# + # remove spaces from return # + ############################# + ENV="$(echo -e "${ENV}" | tr -d '[:space:]')" + ################################ + # Get the env to add to string # + ################################ + ENV="$(echo "${ENV}" | cut -d'"' -f2)" + # echo "ENV:[$ENV]" + ENV_STRING+="--env ${ENV} " + done #################### # Pull in the file # #################### @@ -1558,7 +1606,7 @@ StandardLint() ################################ # Lint the file with the rules # ################################ - STANDARD_LINT_CMD=$(standard "$FILE" 2>&1) + STANDARD_LINT_CMD=$(standard "$ENV_STRING" "$FILE" 2>&1) ####################### # Load the error code # From 76bd3309a51297e0d7685eca23b6998de4f918d9 Mon Sep 17 00:00:00 2001 From: Lucas Gravley <29484535+admiralAwkbar@users.noreply.github.com> Date: Thu, 12 Dec 2019 10:37:08 -0600 Subject: [PATCH 2/2] dont like that rule --- lib/linter.sh | 1 + 1 file changed, 1 insertion(+) diff --git a/lib/linter.sh b/lib/linter.sh index 842f12bd..4b7fff35 100755 --- a/lib/linter.sh +++ b/lib/linter.sh @@ -1555,6 +1555,7 @@ StandardLint() # Get list of all environment variables # ######################################### # Only env vars that are marked as true + # shellcheck disable=SC2207 GET_ENV_ARRAY=($(yq .env "$JAVASCRIPT_LINTER_RULES" |grep true)) #######################