Since we are now trying to have everyone regularly rotate into a on-call reviewer day, and one of the goals of OCR is that you should try to touch all open reviews. However, I'm finding a bunch of things that have already been reviewed quite thoroughly and look much more like we are just waiting for the person to do what was requested and then ask for review again.
In Launchpad, we used Work in Progress to indicate this. I don't see any equivalent on Github (you just have whether the current PR is open or closed). I'm a little concerned that just Closing a request is going to make it easy for the person who submitted it forget about it. However, I also don't think we want all reviewers to have to poll through a large backlog every day. I suppose a meta question exists, why do we have such a huge pile of things that have been reviewed but not actually responded to by the original person? Also, I do think we want to follow our old Rietveld behavior, where for each comment a reviewer made, the submitter can respond (even if just with "Done"). I realize this generates a lot of email noise, but it means that any reviewer can come along and see what has been addressed and what hasn't. Or at least follow along with the conversation. Thoughts? Is Closed to big of a hammer. Is there something else in our process that we need to focus on? John =:->
-- Juju-dev mailing list [email protected] Modify settings or unsubscribe at: https://lists.ubuntu.com/mailman/listinfo/juju-dev
