actions/lint
1
0
Fork 0
mirror of https://github.com/super-linter/super-linter.git synced 2024-12-22 12:42:09 -05:00
Code Issues Projects Releases Packages Wiki Activity Actions 1

Merge pull request #769 from ferrarimarco/remove-read-only-change-flag

Browse source 
Remove READ_ONLY_CHANGE_FLAG flag and fix .editorconfig and shfmt tests
This commit is contained in:
Lukas Gravley 2020-09-28 08:35:58 -05:00 committed by GitHub
commit 6482f45462
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
6 changed files with 5 additions and 160 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
.automation/test/editorconfig-checker/.editorconfig
View file

@ -1,3 +1,6 @@
# top-most EditorConfig file
root = true
[*.ext] [*.ext]
indent_style = space indent_style = space
indent_size = 4 indent_size = 4

2
.automation/test/shell_shfmt/.editorconfig
View file

@ -1,6 +1,6 @@
# top-most EditorConfig file # top-most EditorConfig file
root = true root = true
[*.ext] [*.sh]
indent_style = space indent_style = space
indent_size = 4 indent_size = 4

0
.automation/test/shell_shfmt/shell_shfmt_bad_1.sh → .automation/test/shell_shfmt/shell-shfmt_bad_1.sh
View file

0
.automation/test/shell_shfmt/shell_shfmt_good_1.sh → .automation/test/shell_shfmt/shell-shfmt_good_1.sh
View file

158
lib/buildFileList.sh
View file

@ -96,10 +96,6 @@ function BuildFileList() {
# Append the file to the array # # Append the file to the array #
################################ ################################
FILE_ARRAY_BASH+=("${FILE}") FILE_ARRAY_BASH+=("${FILE}")
##########################################################
# Set the READ_ONLY_CHANGE_FLAG since this could be exec #
##########################################################
READ_ONLY_CHANGE_FLAG=1
######################### #########################
# Get the CLOJURE files # # Get the CLOJURE files #
@ -110,10 +106,6 @@ function BuildFileList() {
# Append the file to the array # # Append the file to the array #
################################ ################################
FILE_ARRAY_CLOJURE+=("${FILE}") FILE_ARRAY_CLOJURE+=("${FILE}")
##########################################################
# Set the READ_ONLY_CHANGE_FLAG since this could be exec #
##########################################################
READ_ONLY_CHANGE_FLAG=1
######################## ########################
# Get the COFFEE files # # Get the COFFEE files #
@ -123,10 +115,6 @@ function BuildFileList() {
# Append the file to the array # # Append the file to the array #
################################ ################################
FILE_ARRAY_COFFEESCRIPT+=("${FILE}") FILE_ARRAY_COFFEESCRIPT+=("${FILE}")
##########################################################
# Set the READ_ONLY_CHANGE_FLAG since this could be exec #
##########################################################
READ_ONLY_CHANGE_FLAG=1
######################## ########################
# Get the CSHARP files # # Get the CSHARP files #
@ -136,10 +124,6 @@ function BuildFileList() {
# Append the file to the array # # Append the file to the array #
################################ ################################
FILE_ARRAY_CSHARP+=("${FILE}") FILE_ARRAY_CSHARP+=("${FILE}")
##########################################################
# Set the READ_ONLY_CHANGE_FLAG since this could be exec #
##########################################################
READ_ONLY_CHANGE_FLAG=1
##################### #####################
# Get the CSS files # # Get the CSS files #
@ -150,10 +134,6 @@ function BuildFileList() {
# Append the file to the array # # Append the file to the array #
################################ ################################
FILE_ARRAY_CSS+=("${FILE}") FILE_ARRAY_CSS+=("${FILE}")
##########################################################
# Set the READ_ONLY_CHANGE_FLAG since this could be exec #
##########################################################
READ_ONLY_CHANGE_FLAG=1
###################### ######################
# Get the DART files # # Get the DART files #
@ -163,10 +143,6 @@ function BuildFileList() {
# Append the file to the array # # Append the file to the array #
################################ ################################
FILE_ARRAY_DART+=("${FILE}") FILE_ARRAY_DART+=("${FILE}")
##########################################################
# Set the READ_ONLY_CHANGE_FLAG since this could be exec #
##########################################################
READ_ONLY_CHANGE_FLAG=1
######################## ########################
# Get the DOCKER files # # Get the DOCKER files #
@ -176,10 +152,6 @@ function BuildFileList() {
# Append the file to the array # # Append the file to the array #
################################ ################################
FILE_ARRAY_DOCKER+=("${FILE}") FILE_ARRAY_DOCKER+=("${FILE}")
##########################################################
# Set the READ_ONLY_CHANGE_FLAG since this could be exec #
##########################################################
READ_ONLY_CHANGE_FLAG=1
##################### #####################
# Get the ENV files # # Get the ENV files #
@ -189,10 +161,6 @@ function BuildFileList() {
# Append the file to the array # # Append the file to the array #
################################ ################################
FILE_ARRAY_ENV+=("${FILE}") FILE_ARRAY_ENV+=("${FILE}")
##########################################################
# Set the READ_ONLY_CHANGE_FLAG since this could be exec #
##########################################################
READ_ONLY_CHANGE_FLAG=1
######################## ########################
# Get the Golang files # # Get the Golang files #
@ -202,10 +170,6 @@ function BuildFileList() {
# Append the file to the array # # Append the file to the array #
################################ ################################
FILE_ARRAY_GO+=("${FILE}") FILE_ARRAY_GO+=("${FILE}")
##########################################################
# Set the READ_ONLY_CHANGE_FLAG since this could be exec #
##########################################################
READ_ONLY_CHANGE_FLAG=1
######################## ########################
# Get the GROOVY files # # Get the GROOVY files #
@ -216,10 +180,6 @@ function BuildFileList() {
# Append the file to the array # # Append the file to the array #
################################ ################################
FILE_ARRAY_GROOVY+=("$FILE") FILE_ARRAY_GROOVY+=("$FILE")
##########################################################
# Set the READ_ONLY_CHANGE_FLAG since this could be exec #
##########################################################
READ_ONLY_CHANGE_FLAG=1
###################### ######################
# Get the HTML files # # Get the HTML files #
@ -229,11 +189,6 @@ function BuildFileList() {
# Append the file to the array # # Append the file to the array #
##############################p## ##############################p##
FILE_ARRAY_HTML+=("${FILE}") FILE_ARRAY_HTML+=("${FILE}")
##########################################################
# Set the READ_ONLY_CHANGE_FLAG since this could be exec #
##########################################################
READ_ONLY_CHANGE_FLAG=1
###################### ######################
# Get the Java files # # Get the Java files #
@ -243,10 +198,6 @@ function BuildFileList() {
# Append the file to the array # # Append the file to the array #
################################ ################################
FILE_ARRAY_JAVA+=("${FILE}") FILE_ARRAY_JAVA+=("${FILE}")
##########################################################
# Set the READ_ONLY_CHANGE_FLAG since this could be exec #
##########################################################
READ_ONLY_CHANGE_FLAG=1
############################ ############################
# Get the JavaScript files # # Get the JavaScript files #
@ -257,10 +208,7 @@ function BuildFileList() {
################################ ################################
FILE_ARRAY_JAVASCRIPT_ES+=("${FILE}") FILE_ARRAY_JAVASCRIPT_ES+=("${FILE}")
FILE_ARRAY_JAVASCRIPT_STANDARD+=("${FILE}") FILE_ARRAY_JAVASCRIPT_STANDARD+=("${FILE}")
##########################################################
# Set the READ_ONLY_CHANGE_FLAG since this could be exec #
##########################################################
READ_ONLY_CHANGE_FLAG=1
##################### #####################
# Get the JSX files # # Get the JSX files #
##################### #####################
@ -309,20 +257,12 @@ function BuildFileList() {
################################ ################################
FILE_ARRAY_STATES+=("${FILE}") FILE_ARRAY_STATES+=("${FILE}")
fi fi
##########################################################
# Set the READ_ONLY_CHANGE_FLAG since this could be exec #
##########################################################
READ_ONLY_CHANGE_FLAG=1
elif [ "${FILE_TYPE}" == "jsx" ]; then elif [ "${FILE_TYPE}" == "jsx" ]; then
################################ ################################
# Append the file to the array # # Append the file to the array #
################################ ################################
FILE_ARRAY_JSX+=("${FILE}") FILE_ARRAY_JSX+=("${FILE}")
##########################################################
# Set the READ_ONLY_CHANGE_FLAG since this could be exec #
##########################################################
READ_ONLY_CHANGE_FLAG=1
######################## ########################
# Get the KOTLIN files # # Get the KOTLIN files #
@ -332,10 +272,6 @@ function BuildFileList() {
# Append the file to the array # # Append the file to the array #
################################ ################################
FILE_ARRAY_KOTLIN+=("${FILE}") FILE_ARRAY_KOTLIN+=("${FILE}")
##########################################################
# Set the READ_ONLY_CHANGE_FLAG since this could be exec #
##########################################################
READ_ONLY_CHANGE_FLAG=1
##################### #####################
# Get the LUA files # # Get the LUA files #
@ -345,10 +281,6 @@ function BuildFileList() {
# Append the file to the array # # Append the file to the array #
################################ ################################
FILE_ARRAY_LUA+=("$FILE") FILE_ARRAY_LUA+=("$FILE")
##########################################################
# Set the READ_ONLY_CHANGE_FLAG since this could be exec #
##########################################################
READ_ONLY_CHANGE_FLAG=1
####################### #######################
# Get the LaTeX files # # Get the LaTeX files #
@ -358,11 +290,6 @@ function BuildFileList() {
# Append the file to the array # # Append the file to the array #
################################ ################################
FILE_ARRAY_LATEX+=("${FILE}") FILE_ARRAY_LATEX+=("${FILE}")
##########################################################
# Set the READ_ONLY_CHANGE_FLAG since this could be exec #
##########################################################
READ_ONLY_CHANGE_FLAG=1
########################## ##########################
# Get the MARKDOWN files # # Get the MARKDOWN files #
@ -384,10 +311,6 @@ function BuildFileList() {
FILE_ARRAY_PHP_PHPCS+=("${FILE}") FILE_ARRAY_PHP_PHPCS+=("${FILE}")
FILE_ARRAY_PHP_PHPSTAN+=("${FILE}") FILE_ARRAY_PHP_PHPSTAN+=("${FILE}")
FILE_ARRAY_PHP_PSALM+=("${FILE}") FILE_ARRAY_PHP_PSALM+=("${FILE}")
##########################################################
# Set the READ_ONLY_CHANGE_FLAG since this could be exec #
##########################################################
READ_ONLY_CHANGE_FLAG=1
###################### ######################
# Get the PERL files # # Get the PERL files #
@ -398,10 +321,6 @@ function BuildFileList() {
# Append the file to the array # # Append the file to the array #
################################ ################################
FILE_ARRAY_PERL+=("${FILE}") FILE_ARRAY_PERL+=("${FILE}")
##########################################################
# Set the READ_ONLY_CHANGE_FLAG since this could be exec #
##########################################################
READ_ONLY_CHANGE_FLAG=1
############################ ############################
# Get the Powershell files # # Get the Powershell files #
@ -411,10 +330,6 @@ function BuildFileList() {
# Append the file to the array # # Append the file to the array #
################################ ################################
FILE_ARRAY_POWERSHELL+=("${FILE}") FILE_ARRAY_POWERSHELL+=("${FILE}")
##########################################################
# Set the READ_ONLY_CHANGE_FLAG since this could be exec #
##########################################################
READ_ONLY_CHANGE_FLAG=1
################################# #################################
# Get the PROTOCOL BUFFER files # # Get the PROTOCOL BUFFER files #
@ -424,10 +339,6 @@ function BuildFileList() {
# Append the file to the array # # Append the file to the array #
################################ ################################
FILE_ARRAY_PROTOBUF+=("${FILE}") FILE_ARRAY_PROTOBUF+=("${FILE}")
##########################################################
# Set the READ_ONLY_CHANGE_FLAG since this could be exec #
##########################################################
READ_ONLY_CHANGE_FLAG=1
######################## ########################
# Get the PYTHON files # # Get the PYTHON files #
@ -438,10 +349,6 @@ function BuildFileList() {
################################ ################################
FILE_ARRAY_PYTHON_PYLINT+=("${FILE}") FILE_ARRAY_PYTHON_PYLINT+=("${FILE}")
FILE_ARRAY_PYTHON_FLAKE8+=("${FILE}") FILE_ARRAY_PYTHON_FLAKE8+=("${FILE}")
##########################################################
# Set the READ_ONLY_CHANGE_FLAG since this could be exec #
##########################################################
READ_ONLY_CHANGE_FLAG=1
###################### ######################
# Get the RAKU files # # Get the RAKU files #
@ -453,10 +360,6 @@ function BuildFileList() {
# Append the file to the array # # Append the file to the array #
################################ ################################
FILE_ARRAY_RAKU+=("${FILE}") FILE_ARRAY_RAKU+=("${FILE}")
##########################################################
# Set the READ_ONLY_CHANGE_FLAG since this could be exec #
##########################################################
READ_ONLY_CHANGE_FLAG=1
#################### ####################
# Get the R files # # Get the R files #
@ -466,10 +369,6 @@ function BuildFileList() {
# Append the file to the array # # Append the file to the array #
################################ ################################
FILE_ARRAY_R+=("${FILE}") FILE_ARRAY_R+=("${FILE}")
##########################################################
# Set the READ_ONLY_CHANGE_FLAG since this could be exec #
##########################################################
READ_ONLY_CHANGE_FLAG=1
###################### ######################
# Get the RUBY files # # Get the RUBY files #
@ -479,10 +378,6 @@ function BuildFileList() {
# Append the file to the array # # Append the file to the array #
################################ ################################
FILE_ARRAY_RUBY+=("${FILE}") FILE_ARRAY_RUBY+=("${FILE}")
##########################################################
# Set the READ_ONLY_CHANGE_FLAG since this could be exec #
##########################################################
READ_ONLY_CHANGE_FLAG=1
########################### ###########################
# Get the SNAKEMAKE files # # Get the SNAKEMAKE files #
@ -492,10 +387,6 @@ function BuildFileList() {
# Append the file to the array # # Append the file to the array #
################################ ################################
FILE_ARRAY_SNAKEMAKE+=("${FILE}") FILE_ARRAY_SNAKEMAKE+=("${FILE}")
##########################################################
# Set the READ_ONLY_CHANGE_FLAG since this could be exec #
##########################################################
READ_ONLY_CHANGE_FLAG=1
##################### #####################
# Get the SQL files # # Get the SQL files #
@ -505,10 +396,6 @@ function BuildFileList() {
# Append the file to the array # # Append the file to the array #
##############################p## ##############################p##
FILE_ARRAY_SQL+=("${FILE}") FILE_ARRAY_SQL+=("${FILE}")
##########################################################
# Set the READ_ONLY_CHANGE_FLAG since this could be exec #
##########################################################
READ_ONLY_CHANGE_FLAG=1
########################### ###########################
# Get the Terraform files # # Get the Terraform files #
@ -519,10 +406,6 @@ function BuildFileList() {
################################ ################################
FILE_ARRAY_TERRAFORM+=("${FILE}") FILE_ARRAY_TERRAFORM+=("${FILE}")
FILE_ARRAY_TERRAFORM_TERRASCAN+=("${FILE}") FILE_ARRAY_TERRAFORM_TERRASCAN+=("${FILE}")
##########################################################
# Set the READ_ONLY_CHANGE_FLAG since this could be exec #
##########################################################
READ_ONLY_CHANGE_FLAG=1
############################ ############################
# Get the TypeScript files # # Get the TypeScript files #
@ -533,10 +416,6 @@ function BuildFileList() {
################################ ################################
FILE_ARRAY_TYPESCRIPT_ES+=("${FILE}") FILE_ARRAY_TYPESCRIPT_ES+=("${FILE}")
FILE_ARRAY_TYPESCRIPT_STANDARD+=("${FILE}") FILE_ARRAY_TYPESCRIPT_STANDARD+=("${FILE}")
##########################################################
# Set the READ_ONLY_CHANGE_FLAG since this could be exec #
##########################################################
READ_ONLY_CHANGE_FLAG=1
##################### #####################
# Get the TSX files # # Get the TSX files #
@ -546,10 +425,6 @@ function BuildFileList() {
# Append the file to the array # # Append the file to the array #
################################ ################################
FILE_ARRAY_TSX+=("${FILE}") FILE_ARRAY_TSX+=("${FILE}")
##########################################################
# Set the READ_ONLY_CHANGE_FLAG since this could be exec #
##########################################################
READ_ONLY_CHANGE_FLAG=1
##################### #####################
# Get the XML files # # Get the XML files #
@ -559,10 +434,6 @@ function BuildFileList() {
# Append the file to the array # # Append the file to the array #
################################ ################################
FILE_ARRAY_XML+=("${FILE}") FILE_ARRAY_XML+=("${FILE}")
##########################################################
# Set the READ_ONLY_CHANGE_FLAG since this could be exec #
##########################################################
READ_ONLY_CHANGE_FLAG=1
################################ ################################
# Get the CLOUDFORMATION files # # Get the CLOUDFORMATION files #
@ -572,10 +443,6 @@ function BuildFileList() {
# Append the file to the array # # Append the file to the array #
################################ ################################
FILE_ARRAY_YAML+=("${FILE}") FILE_ARRAY_YAML+=("${FILE}")
##########################################################
# Set the READ_ONLY_CHANGE_FLAG since this could be exec #
##########################################################
READ_ONLY_CHANGE_FLAG=1
##################################### #####################################
# Check if the file is CFN template # # Check if the file is CFN template #
@ -585,10 +452,6 @@ function BuildFileList() {
# Append the file to the array # # Append the file to the array #
################################ ################################
FILE_ARRAY_CLOUDFORMATION+=("${FILE}") FILE_ARRAY_CLOUDFORMATION+=("${FILE}")
##########################################################
# Set the READ_ONLY_CHANGE_FLAG since this could be exec #
##########################################################
READ_ONLY_CHANGE_FLAG=1
fi fi
############################################ ############################################
@ -612,8 +475,6 @@ function BuildFileList() {
fi fi
done done
export READ_ONLY_CHANGE_FLAG # Workaround SC2034
######################################### #########################################
# Need to switch back to branch of code # # Need to switch back to branch of code #
######################################### #########################################
@ -681,10 +542,6 @@ function CheckFileType() {
# Append the file to the array # # Append the file to the array #
################################ ################################
FILE_ARRAY_BASH+=("${FILE}") FILE_ARRAY_BASH+=("${FILE}")
##########################################################
# Set the READ_ONLY_CHANGE_FLAG since this could be exec #
##########################################################
READ_ONLY_CHANGE_FLAG=1
elif [[ ${GET_FILE_TYPE_CMD} == *"Ruby script"* ]]; then elif [[ ${GET_FILE_TYPE_CMD} == *"Ruby script"* ]]; then
####################### #######################
# It is a Ruby script # # It is a Ruby script #
@ -695,19 +552,11 @@ function CheckFileType() {
# Append the file to the array # # Append the file to the array #
################################ ################################
FILE_ARRAY_RUBY+=("${FILE}") FILE_ARRAY_RUBY+=("${FILE}")
##########################################################
# Set the READ_ONLY_CHANGE_FLAG since this could be exec #
##########################################################
READ_ONLY_CHANGE_FLAG=1
else else
############################ ############################
# Extension was not found! # # Extension was not found! #
############################ ############################
warn "Failed to get filetype for:[${FILE}]!" warn "Failed to get filetype for:[${FILE}]!"
##########################################################
# Set the READ_ONLY_CHANGE_FLAG since this could be exec #
##########################################################
READ_ONLY_CHANGE_FLAG=1
fi fi
} }
################################################################################ ################################################################################
@ -790,11 +639,6 @@ function PopulateShellScriptsList() {
if IsValidShellScript "${FILE}"; then if IsValidShellScript "${FILE}"; then
debug "Adding ${FILE} to shell script files list" debug "Adding ${FILE} to shell script files list"
FILE_ARRAY_BASH+=("${FILE}") FILE_ARRAY_BASH+=("${FILE}")
##########################################################
# Set the READ_ONLY_CHANGE_FLAG since this could be exec #
##########################################################
READ_ONLY_CHANGE_FLAG=1
fi fi
done done

2
lib/linter.sh
View file

@ -263,8 +263,6 @@ ERROR_ON_MISSING_EXEC_BIT="${ERROR_ON_MISSING_EXEC_BIT:-false}" # Default to rep
export ERROR_ON_MISSING_EXEC_BIT export ERROR_ON_MISSING_EXEC_BIT
RAW_FILE_ARRAY=() # Array of all files that were changed RAW_FILE_ARRAY=() # Array of all files that were changed
export RAW_FILE_ARRAY # Workaround SC2034 export RAW_FILE_ARRAY # Workaround SC2034
READ_ONLY_CHANGE_FLAG=0 # Flag set to 1 if files changed are not txt or md
export READ_ONLY_CHANGE_FLAG # Workaround SC2034
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
export TEST_CASE_FOLDER # Workaround SC2034 export TEST_CASE_FOLDER # Workaround SC2034
WARNING_ARRAY_TEST=() # Array of warning linters that did not have an expected test result. WARNING_ARRAY_TEST=() # Array of warning linters that did not have an expected test result.