Re: [PATCH 1/2] t3301-notes: check editor isn't invoked for empty notes add

From: Junio C Hamano <gitster@pobox.com>
Date: Jul 25, 2024, 17:10 UTC
Message-ID: <xmqqv80tfldo.fsf@gitster.g>
In-Reply-To: <5fdccb84-1e60-477e-99d9-b22e3ef83f54@app.fastmail.com>

"Kristoffer Haugsbakk" <code@khaugsbakk.name> writes:

> Introducing a regression test in one patch and then fixing the bug (and
> making the test pass) in the next patch is a style that some prefer. But
> I have received feedback before that it’s best to avoid that.

Especially for something very small like this topic, yes.