fix: make license flag required

faiq · faiq · commit 9b73715d3eec · 2024-08-14T02:41:00.000-07:00
diff --git a/hack/tools/sync-helm-values/sync-values.go b/hack/tools/sync-helm-values/sync-values.go
@@ -70,8 +70,8 @@ func main() {
 		fmt.Println("failed to ensure full path for argument", err.Error())
 		os.Exit(1)
 	}
-	if kustomizeDir == "" || helmChartDir == "" {
-		fmt.Println("-helm-chart-directory and -kustomize-directory must be set")
+	if kustomizeDir == "" || helmChartDir == "" || licenseFile == "" {
+		fmt.Println("-helm-chart-directory -license-file -kustomize-directory must all be set")
 		os.Exit(1)
 	}
 	err = SyncHelmValues(kustomizeDir, helmChartDir, licenseFile)

Original file line number	Diff line number	Diff line change
`@@ -70,8 +70,8 @@ func main() {`
`70`	`70`	`fmt.Println("failed to ensure full path for argument", err.Error())`
`71`	`71`	`os.Exit(1)`
`72`	`72`	`}`
`73`		`- if kustomizeDir == "" \|\| helmChartDir == "" {`
`74`		`- fmt.Println("-helm-chart-directory and -kustomize-directory must be set")`
	`73`	`+ if kustomizeDir == "" \|\| helmChartDir == "" \|\| licenseFile == "" {`
	`74`	`+ fmt.Println("-helm-chart-directory -license-file -kustomize-directory must all be set")`
`75`	`75`	`os.Exit(1)`
`76`	`76`	`}`
`77`	`77`	`err = SyncHelmValues(kustomizeDir, helmChartDir, licenseFile)`