Thread (11 messages) 11 messages, 6 authors, 2025-05-30

Re: [PATCH] doc: sparse-checkout: Fix list markers

From: Junio C Hamano <hidden>
Date: 2025-05-30 13:38:10

Eric Sunshine [off-list ref] writes:
You're welcome to submit an additional patch implementing Kristoffer's
suggestion. Normally, it would make sense to send them both as a
two-patch series, however, I'm not so sure that that would be a good
idea in this case. The reason I question it is that the original patch
is an obvious and uncontroversial fix, however, Kristoffer's
suggestion -- because it is a subjective change -- may get pushback
from reviewers.
I had the same thought.  If I were working on this topic, I'd send
the original as-is with the suggested name fix, let it graduate, and
then after all the dust settled, do enumeration-style changes as a
new and totally independent patch, which may cover enumerations
other than this single instance.

Thanks.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help