2010-06-10
Entry tags:
The Process of Reviewing
We've been seeing a lot of patches submitted lately, which is awesome (June hackathon, ILU ♥). So, now seems a perfect time to talk about reviews :D
( Below the cut: outlining the roles of a dev, a reviewer, and a committer )
( informal comments, full reviews, and what you should look for when looking over a patch )
ETA: Also of interest is![[personal profile]](https://www.dreamwidth.org/img/silk/identity/user.png) yvi's entry Review 101 which explains how to do reviews in more detail, and also gives examples (thanks
yvi's entry Review 101 which explains how to do reviews in more detail, and also gives examples (thanks ![[personal profile]](https://www.dreamwidth.org/img/silk/identity/user.png) cesy for reminding me!)
cesy for reminding me!)
( Below the cut: outlining the roles of a dev, a reviewer, and a committer )
( informal comments, full reviews, and what you should look for when looking over a patch )
ETA: Also of interest is
![[personal profile]](https://www.dreamwidth.org/img/silk/identity/user.png) yvi's entry Review 101 which explains how to do reviews in more detail, and also gives examples (thanks
yvi's entry Review 101 which explains how to do reviews in more detail, and also gives examples (thanks ![[personal profile]](https://www.dreamwidth.org/img/silk/identity/user.png) cesy for reminding me!)
cesy for reminding me!)![[staff profile]](https://www.dreamwidth.org/img/silk/identity/user_staff.png)

