From c541a221bb228431fef90cf049c36be3e62a7f1e Mon Sep 17 00:00:00 2001 From: Eric Nemchik Date: Thu, 30 Jul 2020 15:05:16 -0500 Subject: [PATCH] Ditch empty echos --- .automation/cleanup-docker.sh | 10 ---------- .automation/upload-docker.sh | 12 ------------ lib/buildFileList.sh | 2 -- lib/linter.sh | 8 -------- lib/validation.sh | 1 - lib/worker.sh | 4 ---- 6 files changed, 37 deletions(-) diff --git a/.automation/cleanup-docker.sh b/.automation/cleanup-docker.sh index 12d29c4b..d9eb6312 100755 --- a/.automation/cleanup-docker.sh +++ b/.automation/cleanup-docker.sh @@ -29,11 +29,9 @@ DOCKERFILE_PATH="${DOCKERFILE_PATH}" # Path to the Dockerfile to be uploaded ################################################################################ #### Function Header ########################################################### Header() { - echo info "-------------------------------------------------------" info "----- GitHub Actions remove image from DockerHub ------" info "-------------------------------------------------------" - echo } ################################################################################ #### Function ValidateInput #################################################### @@ -42,11 +40,9 @@ ValidateInput() { ################ # Print header # ################ - echo info "----------------------------------------------" info "Gathering variables..." info "----------------------------------------------" - echo ############################ # Validate GITHUB_WORKSPACE # @@ -127,11 +123,9 @@ LoginToDocker() { ################ # Print header # ################ - echo info "----------------------------------------------" info "Login to DockerHub..." info "----------------------------------------------" - echo ###################### # Login to DockerHub # @@ -161,11 +155,9 @@ RemoveImage() { ################ # Print header # ################ - echo info "----------------------------------------------" info "Removing the DockerFile image:[${IMAGE_REPO}:${IMAGE_VERSION}]" info "----------------------------------------------" - echo ##################################### # Create Token to auth to DockerHub # @@ -220,11 +212,9 @@ RemoveImage() { ################################################################################ #### Function Footer ########################################################### Footer() { - echo info "-------------------------------------------------------" info "The step has completed" info "-------------------------------------------------------" - echo } ################################################################################ ################################## MAIN ######################################## diff --git a/.automation/upload-docker.sh b/.automation/upload-docker.sh index 0510276a..f9557959 100755 --- a/.automation/upload-docker.sh +++ b/.automation/upload-docker.sh @@ -35,11 +35,9 @@ UPDATE_MAJOR_TAG=0 # Flag to deploy the major tag version as ################################################################################ #### Function Header ########################################################### Header() { - echo info "-------------------------------------------------------" info "---- GitHub Actions Upload image to [${REGISTRY}] ----" info "-------------------------------------------------------" - echo } ################################################################################ #### Function ValidateInput #################################################### @@ -48,11 +46,9 @@ ValidateInput() { ################ # Print header # ################ - echo info "----------------------------------------------" info "Gathering variables..." info "----------------------------------------------" - echo ############################# # Validate GITHUB_WORKSPACE # @@ -232,11 +228,9 @@ Authenticate() { ################ # Print header # ################ - echo info "----------------------------------------------" info "Login to ${NAME}..." info "----------------------------------------------" - echo ################### # Auth to service # @@ -266,11 +260,9 @@ BuildImage() { ################ # Print header # ################ - echo info "----------------------------------------------" info "Building the DockerFile image..." info "----------------------------------------------" - echo ################################ # Validate the DOCKERFILE_PATH # @@ -333,11 +325,9 @@ UploadImage() { ################ # Print header # ################ - echo info "----------------------------------------------" info "Uploading the DockerFile image to ${REGISTRY}..." info "----------------------------------------------" - echo ############################################ # Upload the docker image that was created # @@ -428,11 +418,9 @@ UploadImage() { ################################################################################ #### Function Footer ########################################################### Footer() { - echo info "-------------------------------------------------------" info "The step has completed" info "-------------------------------------------------------" - echo } ################################################################################ ################################## MAIN ######################################## diff --git a/lib/buildFileList.sh b/lib/buildFileList.sh index 2f07defd..6c0ba76b 100755 --- a/lib/buildFileList.sh +++ b/lib/buildFileList.sh @@ -69,7 +69,6 @@ function BuildFileList() { ################################################ # Iterate through the array of all files found # ################################################ - echo info "----------------------------------------------" info "Files that have been modified in the commit(s):" for FILE in "${RAW_FILE_ARRAY[@]}"; do @@ -527,7 +526,6 @@ function BuildFileList() { ################ # Footer print # ################ - echo info "----------------------------------------------" info "Successfully gathered list of files..." } diff --git a/lib/linter.sh b/lib/linter.sh index 7bfe9592..062c2df8 100755 --- a/lib/linter.sh +++ b/lib/linter.sh @@ -364,11 +364,9 @@ Header() { ########## # Prints # ########## - echo info "---------------------------------------------" info "--- GitHub Actions Multi Language Linter ----" info "---------------------------------------------" - echo info "---------------------------------------------" info "The Super-Linter source code can be found at:" info " - https://github.com/github/super-linter" @@ -932,13 +930,11 @@ CallStatusAPI() { ################################################################################ #### Function Reports ########################################################## Reports() { - echo info "----------------------------------------------" info "----------------------------------------------" info "Generated reports:" info "----------------------------------------------" info "----------------------------------------------" - echo ################################### # Prints output report if enabled # @@ -958,13 +954,11 @@ Reports() { ################################################################################ #### Function Footer ########################################################### Footer() { - echo info "----------------------------------------------" info "----------------------------------------------" info "The script has completed" info "----------------------------------------------" info "----------------------------------------------" - echo #################################################### # Need to clean up the lanuage array of duplicates # @@ -1028,10 +1022,8 @@ Footer() { ######################## # Footer prints Exit 0 # ######################## - echo notice "All file(s) linted successfully with no errors detected" info "----------------------------------------------" - echo # Successful exit exit 0 } diff --git a/lib/validation.sh b/lib/validation.sh index 1cc7517d..6e4105b0 100755 --- a/lib/validation.sh +++ b/lib/validation.sh @@ -13,7 +13,6 @@ function GetValidationInfo() { ############################################ # Print headers for user provided env vars # ############################################ - echo info "--------------------------------------------" info "Gathering user validation information..." diff --git a/lib/worker.sh b/lib/worker.sh index a2d1c1eb..b5d5988c 100755 --- a/lib/worker.sh +++ b/lib/worker.sh @@ -276,13 +276,11 @@ function TestCodebase() { ################ # print header # ################ - echo info "----------------------------------------------" info "----------------------------------------------" info "Testing Codebase [${FILE_TYPE}] files..." info "----------------------------------------------" info "----------------------------------------------" - echo ##################################### # Validate we have linter installed # @@ -562,11 +560,9 @@ function RunTestCases() { ################# # Header prints # ################# - echo info "----------------------------------------------" info "-------------- TEST CASE RUN -----------------" info "----------------------------------------------" - echo ####################### # Test case languages #