Thread (18 messages) flat view 18 messages, 4 authors, 2021-09-23

Re: [PATCH] MyFirstContribution: Document --range-diff option when writing v2

From: Bagas Sanjaya <hidden>
Date: 2021-09-14 03:55:23

On 14/09/21 10.46, Eric Sunshine wrote:
Not so. Anyone who does any serious amount of review on this project
finds it tremendously helpful to have both a prose description of the
changes ("Changes since v1..." plus a link to the previous submission)
and a mechanical range-diff or interdiff. A range-diff (or interdiff)
is especially important to provide reviewers with context which they
might have forgotten since the previous version of a patch series was
posted, which can matter since it's so easy to forget specifics even
about one's own review if enough time has passed or if reviewing a
large number of unrelated submissions. A range-diff or interdiff also
helps a reviewer determine at-a-glance whether or not earlier review
comments have been addressed without having to laboriously re-read
each and every patch.
So will range-diff be mandatory or optional?

-- 
An old man doll... just what I always wanted! - Clara
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help