Re: How to review an entire application?

2010-02-22 Thread skip
Christian> Let me see if I have this right. You have a directory full of Christian> committed code that hasn't been reviewed yet. This code may Christian> span multiple revisions. You want to put that code up for Christian> review. Is that correct? Yup. Christian> There's no

Re: How to review an entire application?

2010-02-22 Thread Christian Hammond
Hi Skip, Your versions of post-review and Review Board sound pretty out-of-date, first of all, so I'd recommend setting aside some time to do an upgrade. Let me see if I have this right. You have a directory full of committed code that hasn't been reviewed yet. This code may span multiple revisio

How to review an entire application?

2010-02-22 Thread skip
I'm still getting familiar with Review Boar. I use it like so: * make a branch * edit/debug/lather/rinse/repeat * post-review * visit url & fill in description, title, assign reviewers One of my colleagues asked a few of us to review a couple small applications written by a n