calvinwan@xxxxxxxxxx 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.
