Re: [PATCH 2/8] pretty: simplify if-else to reduce code duplication

From: Martin Ågren <martin.agren@gmail.com>
Date: Mar 20, 2025, 16:10 UTC
Message-ID: <CAN0heSpHZo=+ZwM5wJXQtFVD5jsMJGu8+KmRVcMDUT_ipgtMRw@mail.gmail.com>
In-Reply-To: <Z9vdQPtmUiuobOP6@pks.im>

Hi Patrick,

Thanks for reviewing!

On Thu, 20 Mar 2025 at 10:18, Patrick Steinhardt <ps@pks.im> wrote:

Show 6 quoted lines

>
> On Wed, Mar 19, 2025 at 08:23:35AM +0100, Martin Ågren wrote:
> > First we look for "auto,", then we try "always,", then we fall back to
>
> Nit: we typically have the body carry enough context so that it makes
> sense even without reading the commit subject.

Thanks. I'll add some context: "After spotting "%C", we first ..."

Martin

$ 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.