Hi Chris,

Review Board doesn't have any way of preventing code from being checked in.
As H M said, you'll need a pre-commit hook for this. I think there's one on
http://reviews.review-boardorg/ somewhere. At some point we may bundle and
document one, but not until we have a solid concept of policy control in the
project, and we don't have that today.

Updating diffs is a manual process, but post-review makes it a lot easier.
Just one command and your review request is created/updated. Some people
automate this with a post-commit hook as well, but again, we don't have an
official one to recommend at this time.


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

On Mon, May 11, 2009 at 8:03 AM, Chris <c.d.whitco...@gmail.com> wrote:

> This discussion was moved from another one that was about windows
> installs.
> I was hoping to use review board as such:
> The main development trunk is stable.
> We want to add something new, so a developer takes a branch to work in
> leaving trunk alone.
> Once the developer is happy with their work then add a review request
> (this may pass first time or need a few iterations)
> Once the code gets the ok the developer moves the code from the branch
> into trunk.
> My main concerns are the process of adding the diffs is quite manual,
> can it be automated?
> Also unless code has been given the OK i dont want it to get into
> trunk, is this possible? i.e. can we stop people committing to trunk,
> or is that more something we need to deal with at a people level not
> an automation level?
> Cheers,
> Chris
> >

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 
For more options, visit this group at 

Reply via email to