Page MenuHomePhabricator

Allow CodeReview revisions to have multiple reviewers
Closed, ResolvedPublic

Description

Currently there's a binary review process for most revisions, where one person (Tim) determines that it's okay or not, but others may have looked at it already and think it's fine. It should be possible to have multiple reviews and review hierarchies in CodeReview. Using the comments feature could hackishly be used in this fashion, but a proper implementation would be better.


Version: unspecified
Severity: enhancement

Details

Reference
bz24380

Event Timeline

bzimport raised the priority of this task from to Medium.Nov 21 2014, 11:08 PM
bzimport set Reference to bz24380.
bzimport added a subscriber: Unknown Object (MLST).

This is a personal pet peeve of mine too. I'm not sure if it'd be feasible to tackle this in one weekend but I'm tagging it with bugsmash anyway so we can at least look at it.

Implemented in r75327. Implementation is pretty bare-bones and needs some beefing up, but it's working :)