Josh Heinrichs <joshiheinrichs@xxxxxxxxx> writes: > The synopsis for `git config unset` mentions two positional arguments: > `<name>` and `<value>`. While the first argument is correct, the second > is not. Users are expected to provide the value via `--value=<value>`. > > Remove the positional argument. The `--value=<value>` option is already > documented correctly, so this is all we need to do to fix the > documentation. > > Signed-off-by: Josh Heinrichs <joshiheinrichs@xxxxxxxxx> > --- > Documentation/git-config.txt | 2 +- > builtin/config.c | 4 ++-- > 2 files changed, 3 insertions(+), 3 deletions(-) Patrick, do you mind to forge your Reviewed-by for this one? Will queue. Thanks, both. > diff --git a/Documentation/git-config.txt b/Documentation/git-config.txt > index 7f81fbbea8..3e420177c1 100644 > --- a/Documentation/git-config.txt > +++ b/Documentation/git-config.txt > @@ -12,7 +12,7 @@ SYNOPSIS > 'git config list' [<file-option>] [<display-option>] [--includes] > 'git config get' [<file-option>] [<display-option>] [--includes] [--all] [--regexp] [--value=<value>] [--fixed-value] [--default=<default>] <name> > 'git config set' [<file-option>] [--type=<type>] [--all] [--value=<value>] [--fixed-value] <name> <value> > -'git config unset' [<file-option>] [--all] [--value=<value>] [--fixed-value] <name> <value> > +'git config unset' [<file-option>] [--all] [--value=<value>] [--fixed-value] <name> > 'git config rename-section' [<file-option>] <old-name> <new-name> > 'git config remove-section' [<file-option>] <name> > 'git config edit' [<file-option>] > diff --git a/builtin/config.c b/builtin/config.c > index 97e4d5f57c..d60e106bd6 100644 > --- a/builtin/config.c > +++ b/builtin/config.c > @@ -19,7 +19,7 @@ static const char *const builtin_config_usage[] = { > N_("git config list [<file-option>] [<display-option>] [--includes]"), > N_("git config get [<file-option>] [<display-option>] [--includes] [--all] [--regexp] [--value=<value>] [--fixed-value] [--default=<default>] <name>"), > N_("git config set [<file-option>] [--type=<type>] [--all] [--value=<value>] [--fixed-value] <name> <value>"), > - N_("git config unset [<file-option>] [--all] [--value=<value>] [--fixed-value] <name> <value>"), > + N_("git config unset [<file-option>] [--all] [--value=<value>] [--fixed-value] <name>"), > N_("git config rename-section [<file-option>] <old-name> <new-name>"), > N_("git config remove-section [<file-option>] <name>"), > N_("git config edit [<file-option>]"), > @@ -43,7 +43,7 @@ static const char *const builtin_config_set_usage[] = { > }; > > static const char *const builtin_config_unset_usage[] = { > - N_("git config unset [<file-option>] [--all] [--value=<value>] [--fixed-value] <name> <value>"), > + N_("git config unset [<file-option>] [--all] [--value=<value>] [--fixed-value] <name>"), > NULL > };