From: SZEDER Gábor <szeder.dev@gmail.com>
Date: Fri, 18 Oct 2019 16:31:16 GMT
Subject: Re: [PATCH v4 06/17] sparse-checkout: create 'disable' subcommand
Message-ID: <20191018163116.GH29845@szeder.dev>
In-Reply-To: <55f95f290ef77fa402f3e73f9c3045177bd70e84.1571147765.git.gitgitgadget@gmail.com>

On Tue, Oct 15, 2019 at 01:55:53PM +0000, Derrick Stolee via GitGitGadget wrote:
> diff --git a/builtin/sparse-checkout.c b/builtin/sparse-checkout.c
> index b747b78d34..78a80ce119 100644
> --- a/builtin/sparse-checkout.c
> +++ b/builtin/sparse-checkout.c
> @@ -8,7 +8,7 @@
>  #include "strbuf.h"
>  
>  static char const * const builtin_sparse_checkout_usage[] = {
> -	N_("git sparse-checkout [init|list|set] <options>"),
> +	N_("git sparse-checkout [init|list|set|disable] <options>"),

In the synopsis [] means optional, but I think the subcommands are not
optional; however, the options are:

  git sparse-checkout (init|list|set|disable) [<options>]

Perhaps it would be even better if each subcommand had its own
synopsis line in the short help, see e.g. 'git notes -h' or 'git
remote -h'.