From: Kristoffer Haugsbakk <kristofferhaugsbakk@fastmail.com>
Date: Tue, 22 Jul 2025 08:41:02 GMT
Subject: Re: [PATCH] rev-list: update a NEEDSWORK comment
Message-ID: <1599cd31-436c-4ac7-bbef-be7dfbff00a3@app.fastmail.com>
In-Reply-To: <xmqqecu9w4a5.fsf@gitster.g>

On Tue, Jul 22, 2025, at 00:46, Junio C Hamano wrote:
> The comment was poorly phrased and it wasn't clear what it wanted to
> say.  Strongly discourage this broken pattern to be copied and
> pasted to other code paths.

Why “was”?  Shouldn’t it be “The comment is poorly phrased ... so
[change it]”.  According to SubmittingPatches, “present-tense”.

/nitpick