[tor-bugs] Re: #1736 [Trac]: Trac should have a "needs code-review" status

["Tor Bug Tracker & Wiki" <torproject-admin@xxxxxxxxxxxxxx>]

#1736: Trac should have a "needs code-review" status
--------------------------+-------------------------------------------------
  Reporter:  nickm        |       Owner:  weasel  
      Type:  enhancement  |      Status:  accepted
  Priority:  normal       |   Milestone:          
 Component:  Trac         |     Version:          
Resolution:               |    Keywords:          
    Parent:               |  
--------------------------+-------------------------------------------------
Changes (by weasel):

  * status:  needs_review => accepted
  * owner:  Sebastian => weasel


Comment:

 currently the (re-)assign review sets state to needs_review and changes
 the owner to the reviewer.

 We could have something that just sets the state without modifying owner,
 but how would we distinguish between 'it's the old owner' and 'this is the
 reviewer'?

 We would need to introduce a second new state.

 Where we have 'needs_reviewer' (owner field meaningless) and the other
 state is 'reviewing' (by the owner).  How about that?

-- 
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/1736#comment:8>
Tor Bug Tracker & Wiki <https://trac.torproject.org/>
The Tor Project: anonymity online