chore: demote HelpWanted

I no longer use HelpWanted except very briefly when I am parsing the
command line arguments.

Remove HelpWwanted from the app itself and make it a short-lived
variable instead.

internal/cli/app.go

@@ -24,7 +24,6 @@ type App struct {
 	CmdName     string
 	Usage       string
 	ExitValue   int
-	HelpWanted  bool
 	QuietWanted bool
 }
 
@@ -61,12 +60,13 @@ func (app *App) Flags(args []string) (string, bool) {
 	}
 	cmdFlags := flag.NewFlagSet("gitmirror", flag.ContinueOnError)
 	cmdFlags.SetOutput(io.Discard)
-	var configFile string
-	var configIsDefault bool
-	cmdFlags.BoolVar(&app.HelpWanted, "help", false, "")
-	cmdFlags.BoolVar(&app.HelpWanted, "h", false, "")
+	var helpWanted bool
+	cmdFlags.BoolVar(&helpWanted, "help", false, "")
+	cmdFlags.BoolVar(&helpWanted, "h", false, "")
 	cmdFlags.BoolVar(&app.QuietWanted, "quiet", false, "")
 	cmdFlags.BoolVar(&app.QuietWanted, "q", false, "")
+	var configFile string
+	var configIsDefault bool
 	cmdFlags.StringVar(&configFile, "config", "", "")
 	cmdFlags.StringVar(&configFile, "c", "", "")
 	err := cmdFlags.Parse(args)
@@ -75,7 +75,7 @@ func (app *App) Flags(args []string) (string, bool) {
 	case err != nil:
 		fmt.Fprintf(os.Stderr, "%s: %s\n%s", app.CmdName, err, app.Usage)
 		app.ExitValue = exitFailure
-	case app.HelpWanted:
+	case helpWanted:
 		fmt.Fprintf(os.Stderr, "%s: use 'help' not '-help' or '-h'\n", app.CmdName)
 		fmt.Fprint(os.Stderr, app.Usage)
 		app.ExitValue = exitFailure