Merge pull request #566 from exercism/default-workspace-freal-this-time

Katrina Owen · web-flow · commit 29a26ee7c971 · 2018-06-21T21:57:06.000-06:00
Actually set the default workspace
diff --git a/cmd/configure.go b/cmd/configure.go
@@ -2,6 +2,7 @@ package cmd
 
 import (
 	"fmt"
+	"path"
 	"text/tabwriter"
 
 	"github.com/exercism/cli/config"
@@ -34,7 +35,7 @@ You can also override certain default settings to suit your preferences.
 			return err
 		}
 		if usrCfg.Workspace == "" {
-			fmt.Println(usrCfg.Home + BinaryName)
+			usrCfg.Workspace = path.Join(usrCfg.Home, path.Base(BinaryName))
 		}
 
 		apiCfg := config.NewEmptyAPIConfig()

Original file line number	Diff line number	Diff line change
`@@ -2,6 +2,7 @@ package cmd`
`2`	`2`
`3`	`3`	`import (`
`4`	`4`	`"fmt"`
	`5`	`+ "path"`
`5`	`6`	`"text/tabwriter"`
`6`	`7`
`7`	`8`	`"github.com/exercism/cli/config"`
`@@ -34,7 +35,7 @@ You can also override certain default settings to suit your preferences.`
`34`	`35`	`return err`
`35`	`36`	`}`
`36`	`37`	`if usrCfg.Workspace == "" {`
`37`		`- fmt.Println(usrCfg.Home + BinaryName)`
	`38`	`+ usrCfg.Workspace = path.Join(usrCfg.Home, path.Base(BinaryName))`
`38`	`39`	`}`
`39`	`40`
`40`	`41`	`apiCfg := config.NewEmptyAPIConfig()`