Cleanup of linter rules prints:

This commit is contained in:
Lucas Gravley 2020-04-27 12:21:38 -05:00
parent 1e768c20f7
commit b616dafd41
3 changed files with 33 additions and 20 deletions

View file

@ -145,7 +145,7 @@ The super-linter allows you to pass the following `ENV` variables to be able to
- **VALIDATE_TERRAFORM** - **VALIDATE_TERRAFORM**
- Default: `true` - Default: `true`
- Flag to enable or disable the linting process of the language - Flag to enable or disable the linting process of the language
- **VERBOSE_OUTPUT** - **ACTIONS_RUNNER_DEBUG**
- Default: `false` - Default: `false`
- Flag to enable additional information about the linter, versions, and additional output - Flag to enable additional information about the linter, versions, and additional output
@ -161,7 +161,7 @@ If you need to disable certain *rules* and *functionality*, you can view [Disabl
The **Docker** container that is built from this repository is located at `https://cloud.docker.com/u/admiralawkbar/repository/docker/admiralawkbar/super-linter` The **Docker** container that is built from this repository is located at `https://cloud.docker.com/u/admiralawkbar/repository/docker/admiralawkbar/super-linter`
## Running Super-Linter locally (troubleshooting/debugging/enhancements) ## Running Super-Linter locally (troubleshooting/debugging/enhancements)
If you find that you need to run super-linter locally, you can follow the documentation at [Running super-linter locally](https://github.com/github/super-linter/blob/master/.github/run-linter-locally.md) If you find that you need to run super-linter locally, you can follow the documentation at [Running super-linter locally](https://github.com/github/super-linter/blob/master/docs/run-linter-locally.md)
### CI/CT/CD ### CI/CT/CD
The **Super-Linter** has *CI/CT/CD* configured utilizing **GitHub** Actions. The **Super-Linter** has *CI/CT/CD* configured utilizing **GitHub** Actions.

View file

@ -94,6 +94,10 @@ Once the container has been downloaded to your local environment, you can then b
- `-e ANSIBLE_DIRECTORY=</path/local/to/codebase/with/ansible>` - `-e ANSIBLE_DIRECTORY=</path/local/to/codebase/with/ansible>`
- Default: `/ansible` - Default: `/ansible`
- Flag to set the root directory for Ansible file location(s) - Flag to set the root directory for Ansible file location(s)
- **ACTIONS_RUNNER_DEBUG**
- `-e ACTIONS_RUNNER_DEBUG=<true|false>`
- Default: `false`
- Flag to enable or disable additional debug info
## Troubleshooting ## Troubleshooting

View file

@ -86,7 +86,7 @@ TEST_CASE_RUN="${TEST_CASE_RUN}" # Boolean to validate only
# Debug Vars # # Debug Vars #
############## ##############
RUN_LOCAL="${RUN_LOCAL}" # Boolean to see if we are running locally RUN_LOCAL="${RUN_LOCAL}" # Boolean to see if we are running locally
VERBOSE_OUTPUT="${VERBOSE_OUTPUT}" # Boolean to see even more info (debug) ACTIONS_RUNNER_DEBUG="${ACTIONS_RUNNER_DEBUG}" # Boolean to see even more info (debug)
################ ################
# Default Vars # # Default Vars #
@ -97,7 +97,7 @@ DEFAULT_WORKSPACE='/tmp/lint' # Default workspace if run
DEFAULT_ANSIBLE_DIRECTORY="$GITHUB_WORKSPACE/ansible" # Default Ansible Directory DEFAULT_ANSIBLE_DIRECTORY="$GITHUB_WORKSPACE/ansible" # Default Ansible Directory
DEFAULT_RUN_LOCAL='false' # Default value for debugging locally DEFAULT_RUN_LOCAL='false' # Default value for debugging locally
DEFAULT_TEST_CASE_RUN='false' # Flag to tell code to run only test cases DEFAULT_TEST_CASE_RUN='false' # Flag to tell code to run only test cases
DEFAULT_VERBOSE_OUTPUT='false' # Default value for debugging output DEFAULT_ACTIONS_RUNNER_DEBUG='false' # Default value for debugging output
RAW_FILE_ARRAY=() # Array of all files that were changed RAW_FILE_ARRAY=() # Array of all files that were changed
READ_ONLY_CHANGE_FLAG=0 # Flag set to 1 if files changed are not txt or md READ_ONLY_CHANGE_FLAG=0 # Flag set to 1 if files changed are not txt or md
TEST_CASE_FOLDER='.automation/test' # Folder for test cases we should always ignore TEST_CASE_FOLDER='.automation/test' # Folder for test cases we should always ignore
@ -219,16 +219,11 @@ GetLinterRules()
FILE_NAME="$1" # Name fo the linter file FILE_NAME="$1" # Name fo the linter file
FILE_LOCATION="$2" # Location of the linter file FILE_LOCATION="$2" # Location of the linter file
################
# print header #
################
echo "----------------------------------------------"
echo "Gathering users linter:[$FILE_NAME] rules from repository, or defaulting..."
##################################### #####################################
# Validate we have the linter rules # # Validate we have the linter rules #
##################################### #####################################
if [ -f "$GITHUB_WORKSPACE/$LINTER_PATH/$FILE_NAME" ]; then if [ -f "$GITHUB_WORKSPACE/$LINTER_PATH/$FILE_NAME" ]; then
echo "----------------------------------------------"
echo "User provided file:[$FILE_NAME], setting rules file..." echo "User provided file:[$FILE_NAME], setting rules file..."
#################################### ####################################
@ -253,8 +248,10 @@ GetLinterRules()
######################################################## ########################################################
# No user default provided, using the template default # # No user default provided, using the template default #
######################################################## ########################################################
if [[ "$ACTIONS_RUNNER_DEBUG" != "false" ]]; then
echo " -> Codebase does NOT have file:[$LINTER_PATH/$FILE_NAME], using Default rules at:[$FILE_LOCATION]" echo " -> Codebase does NOT have file:[$LINTER_PATH/$FILE_NAME], using Default rules at:[$FILE_LOCATION]"
fi fi
fi
} }
################################################################################ ################################################################################
#### Function GetStandardRules ################################################# #### Function GetStandardRules #################################################
@ -378,7 +375,7 @@ LintAnsibleFiles()
exit 1 exit 1
else else
# Success # Success
if [[ "$VERBOSE_OUTPUT" != "false" ]]; then if [[ "$ACTIONS_RUNNER_DEBUG" != "false" ]]; then
# Success # Success
echo "Successfully found binary in system" echo "Successfully found binary in system"
echo "Location:[$VALIDATE_INSTALL_CMD]" echo "Location:[$VALIDATE_INSTALL_CMD]"
@ -624,13 +621,17 @@ GetGitHubVars()
echo "Successfully found:[GITHUB_REPO], value:[$GITHUB_REPO]" echo "Successfully found:[GITHUB_REPO], value:[$GITHUB_REPO]"
fi fi
fi fi
}
################################################################################
#### Function GetValidationInfo ################################################
GetValidationInfo()
{
############################################ ############################################
# Print headers for user provided env vars # # Print headers for user provided env vars #
############################################ ############################################
echo "" echo ""
echo "--------------------------------------------" echo "--------------------------------------------"
echo "Gathering User provided information..." echo "Gathering user validation information..."
########################################### ###########################################
# Skip validation if were running locally # # Skip validation if were running locally #
@ -946,22 +947,22 @@ GetGitHubVars()
############################ ############################
# Get the run verbose flag # # Get the run verbose flag #
############################ ############################
if [ -z "$VERBOSE_OUTPUT" ]; then if [ -z "$ACTIONS_RUNNER_DEBUG" ]; then
################################## ##################################
# No flag passed, set to default # # No flag passed, set to default #
################################## ##################################
VERBOSE_OUTPUT="$DEFAULT_VERBOSE_OUTPUT" ACTIONS_RUNNER_DEBUG="$DEFAULT_ACTIONS_RUNNER_DEBUG"
fi fi
############################### ###############################
# Convert string to lowercase # # Convert string to lowercase #
############################### ###############################
VERBOSE_OUTPUT=$(echo "$VERBOSE_OUTPUT" | awk '{print tolower($0)}') ACTIONS_RUNNER_DEBUG=$(echo "$ACTIONS_RUNNER_DEBUG" | awk '{print tolower($0)}')
################### ###################
# Debug on runner # # Debug on runner #
################### ###################
if [[ "$VERBOSE_OUTPUT" != "false" ]]; then if [[ "$ACTIONS_RUNNER_DEBUG" != "false" ]]; then
echo "--- DEBUG ---" echo "--- DEBUG ---"
echo "---------------------------------------------" echo "---------------------------------------------"
RUNNER=$(whoami) RUNNER=$(whoami)
@ -1749,6 +1750,14 @@ Header
# needed to connect back and update checks # needed to connect back and update checks
GetGitHubVars GetGitHubVars
##########################################
# Get the langugages we need to validate #
##########################################
if [[ "$ACTIONS_RUNNER_DEBUG" != "false" ]]; then
# Get the flags for langugaes to validate
GetValidationInfo
fi
######################## ########################
# Get the linter rules # # Get the linter rules #
######################## ########################
@ -1778,7 +1787,7 @@ GetLinterRules "$TERRAFORM_FILE_NAME" "$TERRAFORM_LINTER_RULES"
################################# #################################
# Check if were in verbose mode # # Check if were in verbose mode #
################################# #################################
if [[ "$VERBOSE_OUTPUT" != "false" ]]; then if [[ "$ACTIONS_RUNNER_DEBUG" != "false" ]]; then
################################## ##################################
# Get and print all version info # # Get and print all version info #
################################## ##################################