Re: [PATCH 01/34] CodingGuidelines: update and clarify command-line conventions

From: Junio C Hamano <gitster@pobox.com>
Date: Sep 7, 2022, 20:19 UTC
Message-ID: <xmqqedwnszsz.fsf@gitster.g>
In-Reply-To: <patch-01.34-b8cbd443987-20220902T092734Z-avarab@gmail.com>

Ævar Arnfjörð Bjarmason  <avarab@gmail.com> writes:

> + Use spacing around "|" token(s), but not on the inside of "[]()"
> + tokens:
> +   Do: [-q | --quiet]
> +   Don't: [-q|--quiet]

It is unclear what "but not on ..." in the description and it is still unclear with the examples. Does this mean

	Don't: ( -q | --quiet )

In other words, "on the inside of" above is meant to forbid spaces immediately after opening or before closing a [] or () pair?

Other than that, I think these are good guidelines to have.

Thanks.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.