Happy user

Dear Review Board developers, I would like to thank you! Review Board is well programmed and easily extensible. We are using a customized version since over a year and had several thousand successful reviews with less pain :) I implemented post review functionality for both TFS and Perforce in

RE: Happy user

I am quite interested to hear how you enabled TFS. Please share :) Tim -Original Message- From: reviewboard@googlegroups.com [mailto:reviewbo...@googlegroups.com] On Behalf Of Philipp Henkel Sent: August-27-10 4:34 AM To: reviewboard Subject: Happy user Dear Review Board developers, I

Re: Happy user

My TFS solution is based on the Java command line client. After reducing the number of TFS commands per review request the performance is quite acceptable. Philipp On Aug 27, 12:26 pm, Timothy Pinet tim.pi...@gmail.com wrote: I am quite interested to hear how you enabled TFS. Please share :)

Re: keeping track of large number of files

Christian, I agree with you as well. However, when someone does publish a massive review, I have no recourse. Here are some suggestions (and only suggestions, meant only to get some conversation going): - allow me to put a comment at a file level (i.e. not on a line number within the file). I

RE: Happy user

Can you elaborate on your exact procedure? Perhaps an article in a blog. Thanks, Tim -Original Message- From: reviewboard@googlegroups.com [mailto:reviewbo...@googlegroups.com] On Behalf Of Philipp Henkel Sent: August-27-10 9:29 AM To: reviewboard Subject: Re: Happy user My TFS solution

mercurial-reviewboard - versions?

Hi, I've using mercurial-reviewboard extension and getting some exceptions thrown on various combinations of command lines: hg postreview -o hg postreview -l == AttributeError: 'localrepository' object has no attribute 'findoutgoing' I'm currently using: ReviewBoard 1.0.7

CVS post-review woes

RB, Please help: I am able to get either Got API Error 105 (HTTP code 200): One or more fields had errors Error data: {u'fields': {u'path': [u'Permission denied, please try again.\r\nPermission denied, please try again.\r\nPermission denied (publickey,gssapi-with-mic,password).\r\ncvs

Re: CVS post-review woes

All, I intended the path mirror to match. The fact that they do not is a typo. ... they should both be path :ext:devd...@foo.bar.com/home/cvshttp://ext:nmil...@gpscvs.cybersource.com/home/cvs mirror path :ext:devd...@foo.bar.com/home/cvs cheers, Noel On Fri, Aug 27, 2010 at 1:29 PM,

Issue 1792 in reviewboard: post-review fails to upload diff with only binary files modified

Status: New Owner: Labels: Type-Defect Priority-Medium New issue 1792 by dmitry.murashenkov: post-review fails to upload diff with only binary files modified http://code.google.com/p/reviewboard/issues/detail?id=1792 What version are you running? 1.0.9 What steps will reproduce the