Merge pull request #968 from posit-dev/mm-fix-empty-target-name

Set target name in CLI before deploying
posit-dev · Feb 12, 2024 · 8fd2cf9 · 8fd2cf9
2 parents 860392b + fffb857
commit 8fd2cf9
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 2 deletions.
diff --git a/cmd/publisher/commands/deploy.go b/cmd/publisher/commands/deploy.go
@@ -56,6 +56,7 @@ func (cmd *DeployCmd) Run(args *cli_types.CommonArgs, ctx *cli_types.CLIContext)
 	if err != nil {
 		return err
 	}
+	stateStore.TargetName = cmd.SaveName
 	fmt.Printf("Deploy to server %s using account %s and configuration %s, creating deployment %s\n",
 		stateStore.Account.URL,
 		stateStore.Account.Name,

diff --git a/internal/publish/publish.go b/internal/publish/publish.go
@@ -214,8 +214,6 @@ func (p *defaultPublisher) createDeploymentRecord(
 			}
 		}
 		p.TargetName = p.SaveName
-	} else if p.TargetName == "" {
-		p.TargetName = string(contentID)
 	}
 	return p.writeDeploymentRecord(log)
 }