Thread (1 message) 1 message, 1 author, 2019-12-03

Re: What's cooking in git.git (Dec 2019, #01; Mon, 2)

From: Junio C Hamano <hidden>
Date: 2019-12-03 16:35:01

Denton Liu [off-list ref] writes:
vague commit messages from the first round of review. Note that 4/5 has
a spurious change that should be removed ...
Which was exactly why I did not pick up the last posted one.  In
general, I would not pick up a series when the author is aware of an
issue to be fixed, and more importantly, when the author knows how
to fix that issue.  That way, I do not have to be as intimate with
the issue-and-fix as the author and the reviewer, who came up with
it in their exchange.  That way we can avoid unnecessary mistakes,
trying pick up the latest one and making an incorrect tweak that is
different from what the author and the reviewer agreed to be the
good way forward.

It's a different story when that reviewer is me, of course---in that
corner case, I should be just as familiar with the final issue as
the author would be, so it may save time without risking mixups for
me to add the final finishing touches myself.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help