Skip to content

Commit

Permalink
Nit: Remove the trailing period after printing $KUDO_HOME
Browse files Browse the repository at this point in the history
The trailing period is too easy to pick up when selecting text in your
terminal to copy.
  • Loading branch information
yankcrime committed Sep 10, 2019
1 parent b9438a7 commit e7a8949
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
4 changes: 2 additions & 2 deletions pkg/kudoctl/cmd/init.go
Original file line number Diff line number Diff line change
Expand Up @@ -124,14 +124,14 @@ func (initCmd *initCmd) run() error {
if err := initCmd.initialize(); err != nil {
return fmt.Errorf("error initializing: %s", err)
}
fmt.Fprintf(initCmd.out, "$KUDO_HOME has been configured at %s.\n", Settings.Home)
fmt.Fprintf(initCmd.out, "$KUDO_HOME has been configured at %s\n", Settings.Home)

// initialize server
if !initCmd.clientOnly {
if initCmd.client == nil {
client, err := kube.GetKubeClient(Settings.KubeConfig)
if err != nil {
return fmt.Errorf("could not get kubernetes client: %s", err)
return fmt.Errorf("could not get Kubernetes client: %s", err)
}
initCmd.client = client
}
Expand Down
2 changes: 1 addition & 1 deletion pkg/kudoctl/cmd/init_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -67,7 +67,7 @@ func TestInitCmd_exists(t *testing.T) {
if err := cmd.run(); err == nil {
t.Errorf("expected error: %v", err)
}
expected := "$KUDO_HOME has been configured at /opt.\n" + "Warning: KUDO is already installed in the cluster.\n" +
expected := "$KUDO_HOME has been configured at /opt\n" + "Warning: KUDO is already installed in the cluster.\n" +
"(Use --client-only to suppress this message)"

if !strings.Contains(buf.String(), expected) {
Expand Down

0 comments on commit e7a8949

Please sign in to comment.