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 -- 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 firstname.lastname@example.org To unsubscribe from this group, send email to reviewboard+unsubscr...@googlegroups.com For more options, visit this group at http://groups.google.com/group/reviewboard?hl=en -~----------~----~----~----~------~----~------~--~---