fix helm_versionstring issue and remove unneeded comments

Author: ceelias

bin/helm-include.sh

@@ -61,7 +61,7 @@ function helm2ReleaseCheck {
       log_error "A Helm 2.x release of [$release] already exists"
       log_error "Helm [$HELM_VER_FULL] cannot manage the Helm 2.x release of [$release]"
       exit 1
-    fi  
+    fi
   fi
 }
 
@@ -72,7 +72,7 @@ function helmRepoAdd {
   ## If this is an air gap deployment, do nothing
   if [ "$AIRGAP_DEPLOYMENT" == "true" ]; then
     return 0
-  fi 
+  fi
 
   HELM_FORCE_REPO_UPDATE=${HELM_FORCE_REPO_UPDATE:-true}
   if [[ ! $(helm repo list 2>/dev/null) =~ $repo[[:space:]] ]]; then
@@ -128,7 +128,8 @@ function get_helm_versionstring {
     if [ "$V4M_HELM_USE_LATEST"  == "true" ]; then
        :  # return null string
     else
-       echo "--version $1"
+       # Explicitly use printf instead of echo to avoid quote issues
+       printf -- "--version %s" "$1"
     fi
 
     return

logging/bin/deploy_fluentbit_azmonitor.sh

@@ -3,9 +3,7 @@
 # Copyright © 2020, SAS Institute Inc., Cary, NC, USA.  All Rights Reserved.
 # SPDX-License-Identifier: Apache-2.0
 
-# shellcheck source=/dev/null
 cd "$(dirname "$BASH_SOURCE")/../.." || exit 1
-# shellcheck source=/dev/null
 source logging/bin/common.sh
 
 # Fix SC2155: Declare and assign separately

logging/bin/deploy_fluentbit_k8sevents_opensearch.sh

@@ -4,9 +4,7 @@
 # SPDX-License-Identifier: Apache-2.0
 
 cd "$(dirname "$BASH_SOURCE")/../.." || exit 1
-# shellcheck source=logging/bin/common.sh
 source logging/bin/common.sh
-# shellcheck source=logging/bin/secrets-include.sh
 source logging/bin/secrets-include.sh
 
 this_script=$(basename "$0")