"Matheus Felipe via GitGitGadget" <gitgitgadget@xxxxxxxxx> writes:
> Subject: Re: [PATCH v2] fix: include the type flag in the cli docs
One more (hopefully final) nit. By reading the above alone (which
is what people will see in "git shortlog --no-merges" output), you
can not even guess which subcommand the "fix" is about, or if it is
even a fix to the code, or just a documentation update.
I'd retitle it to
config: correct "--type" option in "git config -h" output
> From: Matheus Felipe <matheusfelipeog@xxxxxxxxxxxxxx>
>
> The usage help for --type option of `git config` is missing `type`
> in the argument placeholder (`<>`). Add it.
Good.
> diff --git a/builtin/config.c b/builtin/config.c
> index 542d8d02b2b..2aea465466b 100644
> --- a/builtin/config.c
> +++ b/builtin/config.c
> @@ -151,7 +151,7 @@ static struct option builtin_config_options[] = {
> OPT_BIT(0, "get-color", &actions, N_("find the color configured: slot [default]"), ACTION_GET_COLOR),
> OPT_BIT(0, "get-colorbool", &actions, N_("find the color setting: slot [stdout-is-tty]"), ACTION_GET_COLORBOOL),
> OPT_GROUP(N_("Type")),
> - OPT_CALLBACK('t', "type", &type, "", N_("value is given this type"), option_parse_type),
> + OPT_CALLBACK('t', "type", &type, N_("type"), N_("value is given this type"), option_parse_type),
Good again.
Thanks. Will queue.
