From: Øystein Walle Date: Wed, 08 Nov 2023 07:53:57 GMT Subject: Re: [PATCH 2/9] for-each-ref: clarify interaction of --omit-empty & --count Message-ID: In-Reply-To: On Tue, 7 Nov 2023 at 20:30, Victoria Dye 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