264: Specify intended reviewers on post-review commandline
- Fixed
- Review Board
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.
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