The New Reviewers List

Benton, Kevin kevin.benton at amd.com
Wed Mar 23 19:48:45 UTC 2005


> On Wed, 2005-03-23 at 10:18 -0800, Benton, Kevin wrote:
> > My other reason for requesting it that way is to keep us from having
to
> > use the list so often.  The benefit of having filename-reviewers@ is
we
> > don't need to have your name memorized :)
> 
> 	Hahahaha. Well, actually, I kinda want people to look at the
list
> frequently. That way, if it changes, people will always be looking at
> the most recent copy. That is, there won't always be mass
notifications
> when it changes, saying "Now people ask so-and-so for reviews on file
> X." :-)

That's my whole point - if the reviewer changes in the role address
model (versus the personal address model), the destination doesn't
change.  They still use the filename-... role account.  It also allows
the people doing the reviews for a particular component of Bugzilla to
see the entire list of reviews being requested rather than some get one
list and others get a different one.  :)  I've thought this through and
used this type of method before - it really works well. :)

The other benefit to us as developers is the lessening of concern about
requesting review from the "right" person.  We know that the lists of
assignees is being maintained and volunteer reviewers can jump in at any
time by doing a search on the reviewer in question.

---
Kevin Benton
Perl/Bugzilla Developer
Advanced Micro Devices
 
The opinions stated in this communication do not necessarily reflect the
view of Advanced Micro Devices and have not been reviewed by management.
This communication may contain sensitive and/or confidential and/or
proprietary information.  Distribution of such information is strictly
prohibited without prior consent of Advanced Micro Devices.  This
communication is for the intended recipient(s) only.  If you have
received this communication in error, please notify the sender, then
destroy any remaining copies of this communication.





More information about the developers mailing list