From: Ævar Arnfjörð Bjarmason <avarab@gmail.com>
Date: Thu, 18 Aug 2022 18:43:22 GMT
Subject: Re: [PATCH v4 07/11] builtin/diagnose.c: create 'git diagnose' builtin
Message-ID: <220818.86edxd8k8q.gmgdl@evledraar.gmail.com>
In-Reply-To: <b58d13325b2eaef32fac89fb2be8e29004560c45.1660335019.git.gitgitgadget@gmail.com>


On Fri, Aug 12 2022, Victoria Dye via GitGitGadget wrote:

> From: Victoria Dye <vdye@github.com>
> [...]

This is correct:

> +'git diagnose' [(-o | --output-directory) <path>] [(-s | --suffix) <format>]

...

> +	N_("git diagnose [-o|--output-directory <path>] [-s|--suffix <format>]"),

But this is not, it's missing () around the short v.s. long option, and
we should have a space surrounding the "|" as well.