From: Kristoffer Haugsbakk <kristofferhaugsbakk@fastmail.com>
Date: Wed, 28 May 2025 08:01:17 GMT
Subject: Re: [PATCH] doc: sparse-checkout: Fix list markers
Message-ID: <4c1f645b-42fb-40b9-987e-08ce9e057e01@app.fastmail.com>
In-Reply-To: <CAPig+cTdN=x2GXGW2p_g_k4jgC0eQmjW5zLsWhJm-PP7eMBTiQ@mail.gmail.com>

On Wed, May 28, 2025, at 09:20, Eric Sunshine wrote:
>> Because then you can use commas freely inside each point.
>
> I'd like to push back on this suggestion, at least in the context of
> this particular patch, the purpose of which is to fix an actual
> problem in the documentation. The sort of improvement you suggest
> probably ought to be done as a separate patch, but let's not make it
> seem as if we are heaping extra work on the patch submitter; it's more
> important to get the fix itself landed.
>
> As such, the patch itself looks obviously correct and is a definite
> improvement as-is. It's not clear why it didn't get picked up; perhaps
> it just fell through the cracks.

Okay, totally reasonable and makes sense IMO.