Clarify git ssh error message (#664)

michaeljguarino · web-flow · commit 4aa44a5797bd · 2025-09-10T18:42:58.000-04:00
This should make it more clear that we currently only support ssh clones for the plural up process
diff --git a/pkg/wkspace/validator.go b/pkg/wkspace/validator.go
@@ -17,9 +17,11 @@ func Preflight() (bool, error) {
 			return true, utils.HighlightError(fmt.Errorf("%s not installed", req))
 		}
 	}
-	fmt.Print("\nTesting if git ssh is properly configured...")
+	fmt.Print("\nTesting if git ssh is properly configured...\n")
 	if err := checkGitSSH(); err != nil {
-		return true, err
+		fmt.Printf("%s\n\n", err.Error())
+		utils.Warn("Please ensure that you have ssh keys set up for git and that you've added them to your ssh agent. You can use `plural crypto ssh-keygen` to create your first ssh keys then upload the public key to your git provider.\n")
+		return true, fmt.Errorf("git ssh is not properly configured")
 	}
 	fmt.Println(" \033[32m (\u2713) \033[0m") // (✔)
 

Original file line number	Diff line number	Diff line change
`@@ -17,9 +17,11 @@ func Preflight() (bool, error) {`
`17`	`17`	`return true, utils.HighlightError(fmt.Errorf("%s not installed", req))`
`18`	`18`	`}`
`19`	`19`	`}`
`20`		`- fmt.Print("\nTesting if git ssh is properly configured...")`
	`20`	`+ fmt.Print("\nTesting if git ssh is properly configured...\n")`
`21`	`21`	`if err := checkGitSSH(); err != nil {`
`22`		`- return true, err`
	`22`	`+ fmt.Printf("%s\n\n", err.Error())`
	`23`	+ utils.Warn("Please ensure that you have ssh keys set up for git and that you've added them to your ssh agent. You can use `plural crypto ssh-keygen` to create your first ssh keys then upload the public key to your git provider.\n")
	`24`	`+ return true, fmt.Errorf("git ssh is not properly configured")`
`23`	`25`	`}`
`24`	`26`	`fmt.Println(" \033[32m (\u2713) \033[0m") // (✔)`
`25`	`27`