On 04/07/2015 01:35 PM, James Bottomley wrote:
If I look at the history, I also see some reviewers dropping out once their concerns and review comments have been addressed (after giving a +1), so the other thing I'd suggest is that instead of erasing the review history on each patch submission, it carries over (at least the -1 and +1) so you don't have to wait a while for a consensus to form (reviewers would, of course, be able to alter their votes at any time). The pressure is thus on the submitter to make the changes to switch every original -1 to a +1.
How would we deal with the scenario where someone leaves a -1 with a comment and then never goes back to check if their concerns were dealt with?
Chris __________________________________________________________________________ OpenStack Development Mailing List (not for usage questions) Unsubscribe: [email protected]?subject:unsubscribe http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
