Last modified: 2011-04-15 19:20:56 UTC

Wikimedia Bugzilla is closed!

Wikimedia has migrated from Bugzilla to Phabricator. Bug reports should be created and updated in Wikimedia Phabricator instead. Please create an account in Phabricator and add your Bugzilla email address to it.
Wikimedia Bugzilla is read-only. If you try to edit or create any bug report in Bugzilla you will be shown an intentional error message.
In order to access the Phabricator task corresponding to a Bugzilla report, just remove "static-" from its URL.
You could still run searches in Bugzilla or access your list of votes but bug reports will obviously not be up-to-date in Bugzilla.
Bug 24380 - Allow CodeReview revisions to have multiple reviewers
Allow CodeReview revisions to have multiple reviewers
Product: MediaWiki extensions
Classification: Unclassified
CodeReview (Other open bugs)
All All
: Normal enhancement (vote)
: ---
Assigned To: Nobody - You can work on this!
Depends on:
  Show dependency treegraph
Reported: 2010-07-15 01:45 UTC by MZMcBride
Modified: 2011-04-15 19:20 UTC (History)
4 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Description MZMcBride 2010-07-15 01:45:27 UTC
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.
Comment 1 Roan Kattouw 2010-09-24 14:49:06 UTC
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.
Comment 2 Roan Kattouw 2010-10-24 18:19:02 UTC
Implemented in r75327. Implementation is pretty bare-bones and needs some beefing up, but it's working :)
Comment 3 Trevor Parscal 2010-10-25 20:18:01 UTC
See for a suggestion on how to lay this out.

Note You need to log in before you can comment on or make changes to this bug.