Re: [PATCH 2/9] for-each-ref: clarify interaction of --omit-empty & --count

From: Øystein Walle <oystwa@gmail.com>
Date: Nov 8, 2023, 07:53 UTC
Message-ID: <CAFaJEquUBYUO=scjHw2qrUyP-4wZJWtdmWAtRrW0mH9x9PbZZw@mail.gmail.com>
In-Reply-To: <e166abeb-3566-4acf-a252-bc493ee37f41@github.com>

On Tue, 7 Nov 2023 at 20:30, Victoria Dye <vdye@github.com> wrote:

> Since the interaction isn't clearly defined at the moment, we could probably
> still update it to work like you're describing here. I'm happy to drop this
> patch and implement your recommendation in a follow-up series. Let me know
> what you think!

Regardless of whether the logic is changed in a follow-up series or not I think the current behavior is worth documenting even if it doesn't exist for much longer in the tree. So I am favor of having this patch as part of this series.

I am also in favor of changing the behavior but that warrants a separate series and discussion.

Øsse

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