Question about setting up Default Reviewers

2009-09-25 Thread Hui Lin

If we want person A to review all the changes made to any class of in
directory D, but also want person B to review changes made to a
particular class C1 in D, how should we define the default
reviewers?

Would the followings work? Or only the first matched reviewer will be
picked?
* Default Reviewer #1: D\/.*
* Default Reviewer #2: D\/C1


--~--~-~--~~~---~--~~
You received this message because you are subscribed to the Google Groups 
reviewboard group.
To post to this group, send email to reviewboard@googlegroups.com
To unsubscribe from this group, send email to 
reviewboard+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/reviewboard?hl=en
-~--~~~~--~~--~--~---



Re: Question about setting up Default Reviewers

2009-09-25 Thread Christian Hammond
Hi,

You can have as many default reviewers as you need, covering redundant parts
of the codebase or referencing the same people/groups multiple times.

The setup you described should work fine.

Christian

-- 
Christian Hammond - chip...@chipx86.com
Review Board - http://www.review-board.org
VMware, Inc. - http://www.vmware.com


On Fri, Sep 25, 2009 at 8:24 AM, Hui Lin hui@bankofamerica.com wrote:


 If we want person A to review all the changes made to any class of in
 directory D, but also want person B to review changes made to a
 particular class C1 in D, how should we define the default
 reviewers?

 Would the followings work? Or only the first matched reviewer will be
 picked?
* Default Reviewer #1: D\/.*
* Default Reviewer #2: D\/C1


 


--~--~-~--~~~---~--~~
You received this message because you are subscribed to the Google Groups 
reviewboard group.
To post to this group, send email to reviewboard@googlegroups.com
To unsubscribe from this group, send email to 
reviewboard+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/reviewboard?hl=en
-~--~~~~--~~--~--~---