Re: [PATCH 00/20] Fix typos

[Eric Sunshine <sunshine@sunshineco.com>]

On Thu, Sep 19, 2024 at 4:42 PM Junio C Hamano <gitster@pobox.com> wrote:
> "Kristoffer Haugsbakk" <kristofferhaugsbakk@fastmail.com> writes:
> > Maybe all of the commits could be combined into one commit with the
> > message “treewide: fix typos”.  But that wasn’t asked for last time.[1]
> > So I dunno.
>
> It depends.  When done carefully to make sure that nothing textually
> conflicts with any topics in flight, a single "treewide" patch is
> slightly nicer.  If nothing conflicts with nothing else, either
> format is fine.  If something have nasty conflicts (e.g., a refactor
> moved the comment with typos far from the original location or even
> to a different file), individual patch form is easier to discard the
> ones that need to wait.
>
> I do not know which case this 20-patch collection falls into.

For what it's worth, I found the submission easier to review as
separate patches because it allowed me to review a small batch, do
something else for a bit, review another batch, do something else,
etc., without losing my place since I deleted the ones I had already
reviewed, so I knew that those remaining in my Inbox were still
pending review.