| commit | 89aa86394e2d3e126f4c34c7032869e8e31671a1 | [log] [tgz] |
|---|---|---|
| author | Dave Borowitz <dborowitz@google.com> | Mon Dec 09 15:11:15 2013 -0800 |
| committer | Dave Borowitz <dborowitz@google.com> | Fri Jan 03 11:03:25 2014 -0800 |
| tree | 450448ec95e405e60152c5f2a83c933eb7bf3cbb | |
| parent | d2d4d1cead4fd778d22042957fc7a53fe343b4a4 [diff] |
Record reviewer state in notes database Stop implicitly adding/removing reviewers by adding an arbitrary PatchSetApprovals with a 0 vote. This is ugly and error-prone, especially when LabelTypes change. Instead, use the existing ReviewerState enum, adding a REMOVED type as a tombstone. When parsing, record these during the walk, and prune all REMOVED tombstones after the fact. Change-Id: Iffb0517eb0162eb6cce66bf36d905a6eb60e75da