Comment #36 on issue 489 by [email protected]: [ ] DON'T SHIP IT button
http://code.google.com/p/reviewboard/issues/detail?id=489
The whole 'karma' idea is fairly common... see e.g. gerrit, Fedora Bodhi, etc. Although I'm not sure that's actually necessary for RB, since we already have issues. If the presence of unresolved issues precluded a review showing as 'ready' (on dashboards particularly, I would say), I think that would generally suffice.
Of course, it would also help if http://code.google.com/p/reviewboard/issues/detail?id=2723 and/or http://code.google.com/p/reviewboard/issues/detail?id=2817 were also implemented.
(More generally, creation of an issue or uploading of a new revision should 'disable' any previous 'Ship It' so that they are not counted on the dashboard. This, plus 2723, would allow reviewers to override open issues, but merge-masters would know when this is happening and be able to verify that such an action is appropriate.)
-- You received this message because you are subscribed to the Google Groups "reviewboard-issues" group. To post to this group, send email to [email protected]. To unsubscribe from this group, send email to [email protected]. For more options, visit this group at http://groups.google.com/group/reviewboard-issues?hl=en.
