Refactor logging in ExecuteAtmosVendorInternal function (#730)

* Refactor logging in ExecuteAtmosVendorInternal function * Refactor logging in ExecuteAtmosVendorInternal function * avoid unnecessary string operations when there are no tags * Refactor logging in ExecuteAtmosVendorInternal function
cloudposse · Oct 20, 2024 · 332a1a9 · 332a1a9
1 parent c866ab5
commit 332a1a9
Showing 1 changed file with 5 additions and 2 deletions.
diff --git a/internal/exec/vendor_utils.go b/internal/exec/vendor_utils.go
@@ -181,8 +181,11 @@ func ExecuteAtmosVendorInternal(
 	var uri string
 	vendorConfigFilePath := path.Dir(vendorConfigFileName)
 
-	u.LogInfo(cliConfig, fmt.Sprintf("Processing vendor config file '%s'", vendorConfigFileName))
-
+	logMessage := fmt.Sprintf("Processing vendor config file '%s'", vendorConfigFileName)
+	if len(tags) > 0 {
+		logMessage = fmt.Sprintf("%s for tags {%s}", logMessage, strings.Join(tags, ", "))
+	}
+	u.LogInfo(cliConfig, logMessage)
 	if len(atmosVendorSpec.Sources) == 0 && len(atmosVendorSpec.Imports) == 0 {
 		return fmt.Errorf("either 'spec.sources' or 'spec.imports' (or both) must be defined in the vendor config file '%s'", vendorConfigFileName)
 	}