From 2e0e42fd9de2fb6db64320d8d883f301eca17c6d Mon Sep 17 00:00:00 2001
From: Kyle Conroy <kyle@conroy.org>
Date: Tue, 20 Jun 2023 12:47:05 -0700
Subject: [PATCH] cmd/sqlc: Show helpful output on missing subcommand

Also use the cobra helpers to avoid printing to stdout directly
---
 internal/cmd/cmd.go | 7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)

diff --git a/internal/cmd/cmd.go b/internal/cmd/cmd.go
index e08ecd7393..3e6c363908 100644
--- a/internal/cmd/cmd.go
+++ b/internal/cmd/cmd.go
@@ -49,7 +49,6 @@ func Do(args []string, stdin io.Reader, stdout io.Writer, stderr io.Writer) int
 	rootCmd.SetIn(stdin)
 	rootCmd.SetOut(stdout)
 	rootCmd.SetErr(stderr)
-	rootCmd.SilenceErrors = true
 
 	ctx := context.Background()
 	if debug.Debug.Trace != "" {
@@ -79,9 +78,9 @@ var versionCmd = &cobra.Command{
 	RunE: func(cmd *cobra.Command, args []string) error {
 		defer trace.StartRegion(cmd.Context(), "version").End()
 		if version == "" {
-			fmt.Printf("%s\n", info.Version)
+			fmt.Fprintf(cmd.OutOrStdout(), "%s\n", info.Version)
 		} else {
-			fmt.Printf("%s\n", version)
+			fmt.Fprintf(cmd.OutOrStdout(), "%s\n", version)
 		}
 		return nil
 	},
@@ -191,7 +190,7 @@ var genCmd = &cobra.Command{
 		dir, name := getConfigPath(stderr, cmd.Flag("file"))
 		output, err := Generate(cmd.Context(), ParseEnv(cmd), dir, name, stderr)
 		if err != nil {
-			return err
+			os.Exit(1)
 		}
 		defer trace.StartRegion(cmd.Context(), "writefiles").End()
 		for filename, source := range output {