| Issue 733: | Include previous approvals in the approvals table | |
| 6 people starred this issue and may be notified of changes. | Back to list |
On my team we occasionally +2 our own patch sets if they've already received a +2 review. Usually this happens when the code's author is asked to make a trivial change (whitespace etc.) before submit. When reviewing such a change in gerrit, it's not obvious at a glance whether a change ever received a +2. You have to read through the comments. I'd like it if gerrit included the history of +2s and other grades in the overview table. I've attached a screenshot to make this concrete, although I don't really care too much how it looks in the UI.
Sep 17, 2010
#1
jessewil...@google.com
Sep 27, 2010
Related to this, the approvals end up getting appended to the commit email, and it would be nice if the prior approvals show up there as well, for the same reasons.
May 20, 2011
(No comment was entered for this change.)
Status:
Accepted
|
|
| ► Sign in to add a comment |