Addresses PR feedback

Azure · Jan 24, 2024 · 71fc302 · 71fc302
1 parent 54f4e9c
commit 71fc302
Show file tree

Hide file tree

Showing 4 changed files with 6 additions and 6 deletions.
diff --git a/cli/azd/cmd/container.go b/cli/azd/cmd/container.go
@@ -468,7 +468,6 @@ func registerCommonDependencies(container *ioc.NestedContainer) {
 	container.MustRegisterSingleton(maven.NewMavenCli)
 	container.MustRegisterSingleton(helm.NewCli)
 	container.MustRegisterSingleton(kustomize.NewCli)
-	container.MustRegisterSingleton(kustomize.NewKustomize)
 	container.MustRegisterSingleton(npm.NewNpmCli)
 	container.MustRegisterSingleton(python.NewPythonCli)
 	container.MustRegisterSingleton(swa.NewSwaCli)
@@ -484,7 +483,7 @@ func registerCommonDependencies(container *ioc.NestedContainer) {
 
 	// Service Targets
 	serviceTargetMap := map[project.ServiceTargetKind]any{
-		"":                               project.NewAppServiceTarget,
+		project.NonSpecifiedTarget:       project.NewAppServiceTarget,
 		project.AppServiceTarget:         project.NewAppServiceTarget,
 		project.AzureFunctionTarget:      project.NewFunctionAppTarget,
 		project.ContainerAppTarget:       project.NewContainerAppTarget,

diff --git a/cli/azd/pkg/helm/cli.go b/cli/azd/pkg/helm/cli.go
@@ -26,12 +26,12 @@ func (cli *Cli) Name() string {
 	return "helm"
 }
 
-// Returns the installation URL to install the K8s CLI
+// Returns the installation URL to install the Helm CLI
 func (cli *Cli) InstallUrl() string {
 	return "https://aka.ms/azure-dev/helm-install"
 }
 
-// Checks whether or not the K8s CLI is installed and available within the PATH
+// Checks whether or not the Helm CLI is installed and available within the PATH
 func (cli *Cli) CheckInstalled(ctx context.Context) error {
 	if err := tools.ToolInPath("helm"); err != nil {
 		return err

diff --git a/cli/azd/pkg/kustomize/cli.go b/cli/azd/pkg/kustomize/cli.go
@@ -26,12 +26,12 @@ func (cli *Cli) Name() string {
 	return "kustomize"
 }
 
-// Returns the installation URL to install the K8s CLI
+// Returns the installation URL to install the Kustomize CLI
 func (cli *Cli) InstallUrl() string {
 	return "https://aka.ms/azure-dev/kustomize-install"
 }
 
-// Checks whether or not the K8s CLI is installed and available within the PATH
+// Checks whether or not the Kustomize CLI is installed and available within the PATH
 func (cli *Cli) CheckInstalled(ctx context.Context) error {
 	if err := tools.ToolInPath("kustomize"); err != nil {
 		return err

diff --git a/cli/azd/pkg/project/service_target.go b/cli/azd/pkg/project/service_target.go
@@ -18,6 +18,7 @@ import (
 type ServiceTargetKind string
 
 const (
+	NonSpecifiedTarget       ServiceTargetKind = ""
 	AppServiceTarget         ServiceTargetKind = "appservice"
 	ContainerAppTarget       ServiceTargetKind = "containerapp"
 	AzureFunctionTarget      ServiceTargetKind = "function"