Re: Review process improvements: drafting a ReviewingPatches doc

[Junio C Hamano <gitster@pobox.com>]

calvinwan@google.com writes:

> In continuation of Emily’s "Review process improvements"[1], I am
> drafting a ReviewingPatches doc to help standardize the review process
> here on the git mailing list and to provide a rubric/checklist for new
> reviewers. In order to do so, I am looking to compile examples and
> input from everyone by asking the list a set of questions biweekly.
> Please contextualize your answer in terms of whether this was a
> maintainer, individual, or drive-by review.

Often people other than the patch contributors themselves find
others' reviews a useful learning opportunity.  I take it that your
"contextualize" request is asking for comments like "As the
maintainer, I found this review by an area expert very helpful
because ...", as well as "I sent this patch and a drive-by typofix
at this URL was not very helpful"?

When soliciting input from the list, it would probably make it
easier for others if you led by example by sending your answers,
to show the level of detail you'd find useful.