logo       
Google Custom Search
    AddThis Social Bookmark Button
-->

Re: Reviewer heirachies ?: msg#00065

Subject: Re: Reviewer heirachies ?
* Peter Miller
| [extensible review policy]
--<snip-snip>--
| 3. add a new "review_policy_command" field to the project config file. 
| It is a command, and Aegis looks at the exist status.
| 
| Substitutions include "$developers" for the list of people who developed
| the change (think "aechown" for why it's plural) and "$reviewers" for
| the folks who have reviewed the change (and passed it) to date.

I think you would also need the list of `all reviewers' to decide whom
to send mail about the new transition.  `$reviewers' in other
substitutions means `all reviewers', so I'd rather use something like
`$previous_reviewers' or `$change_reviewers' here.

| How much do folks want this?

We would use it for implementing different types of review (code
review, `user mode' review), done by different reviewers.  Currently
we're using the shout-across-the-room method, too.

| Anybody want it enough to code it and send it to me?

:-)

R'


<Prev in Thread] Current Thread [Next in Thread>