264: Specify intended reviewers on post-review commandline

ram****@gmai***** (Google Code) (Is this you? Claim this profile.)
Oct. 19, 2007
Please add the ability to specify the intended reviewers as an argument to
post-review.

For example, I want to be able to do this:

$ post-review -m ramesh@vmware.com,christian@vmware.com <12345>

And later, do this:

$ post-review -m davidt@vmware.com,christian@vmware.com <12345>

In the second invocation, post-review would update the reviewers list
appropriately, adding/removing people as needed.
chipx86
#1 chipx86
I realized after we talked that someone already implemented this. We still need to
push the new post-review out, though, and that's going to require supporting a site
defaults file that post-review will use so people don't have to type the path to
reviewboard the first time. Still, if you want to use that and be our guinea pig,
grab the contrib/tools/post-review script from Review Board SVN and you should have
--target-groups and --target-people parameters.
  • +Fixed
  • -Type-Defect
    +Type-Enhancement
    +Component-Scripts