Re: Reviewboard is awesome...

2010-04-01 Thread Christian Hammond
Thanks Ryan :) Glad it's working out well for you. We love hearing this kind of feedback. I'd love to hear how things go as you guys roll it out more. Christian -- Christian Hammond - chip...@chipx86.com Review Board - http://www.reviewboard.org VMware, Inc. - http://www.vmware.com On Wed,

Additional CLI options

2010-04-01 Thread rshelley
I've been able to complete a functional Hudson-Reviewboard plugin (with many thanks). With the core functionality of it done, I'm now looking at how to improve upon it, already. I know there isn't currently a command-line option for this (or may not be one if it doesn't seem to be ideal), but I

Re: Additional CLI options

2010-04-01 Thread Paul
On 01/04/2010, at 14:24, rshelley ryan.shel...@disney.com wrote: I've been able to complete a functional Hudson-Reviewboard plugin (with many thanks). Nice additional params to help with automation. --review-groups=groups (and --review- people=people) to default the review to a

Re: Additional CLI options

2010-04-01 Thread Charles Chan
Hello Ryan, Where can I find more information about this Hudson-Reviewboard plugin? We also use Hudson as our CI server and will be incorporating RB into our process soon. I would learn more about it. Thanks, Charles On Thu, Apr 1, 2010 at 2:24 PM, rshelley ryan.shel...@disney.com wrote: I've

Re: Issue 1581 in reviewboard: Diff veivew error on RBTools 0.2rc1

2010-04-01 Thread reviewboard
Comment #2 on issue 1581 by seagle0128: Diff veivew error on RBTools 0.2rc1 http://code.google.com/p/reviewboard/issues/detail?id=1581 I run with --output-diff. All look good, but on the web page, the issue still exists. I use the reviewboard server on Windows, and the .rej file looks good,

Re: Issue 1581 in reviewboard: Diff veivew error on RBTools 0.2rc1

2010-04-01 Thread reviewboard
Comment #3 on issue 1581 by seagle0128: Diff veivew error on RBTools 0.2rc1 http://code.google.com/p/reviewboard/issues/detail?id=1581 I attached log.zip. Please refer to it for details. Attachments: log.zip 669 bytes -- You received this message because you are listed in the owner

Re: Issue 1581 in reviewboard: Diff veivew error on RBTools 0.2rc1

2010-04-01 Thread reviewboard
Comment #4 on issue 1581 by seagle0128: Diff veivew error on RBTools 0.2rc1 http://code.google.com/p/reviewboard/issues/detail?id=1581 I found some threads. If there is the symbol © in the file, I will get the issue, and in .rej file, it's changed to �. I use Chinese locale. I changed to

Re: Issue 1582 in reviewboard: Dashboard Columns Misaligned

2010-04-01 Thread reviewboard
Updates: Status: Fixed Owner: chipx86 Labels: -Priority-Medium Priority-Critical Djblets Component-Dashboard Milestone-Release1.0.x Comment #1 on issue 1582 by chipx86: Dashboard Columns Misaligned http://code.google.com/p/reviewboard/issues/detail?id=1582 This should be

Re: Issue 1582 in reviewboard: Dashboard Columns Misaligned

2010-04-01 Thread reviewboard
Comment #2 on issue 1582 by tbrezins: Dashboard Columns Misaligned http://code.google.com/p/reviewboard/issues/detail?id=1582 Very close but there is still some minor glitches with column span. For the 2nd image my coworker dug out that there is a colspan when there should not be: td

Re: Issue 1495 in reviewboard: Need a descriptive error message when using --revision-range on unsupported SCMs

2010-04-01 Thread reviewboard
Comment #7 on issue 1495 by nickdowell: Need a descriptive error message when using --revision-range on unsupported SCMs http://code.google.com/p/reviewboard/issues/detail?id=1495 Oh I see, clearly I should have been paying more attention whilst RTFM ;-) -- You received this message because

Re: Issue 1495 in reviewboard: Need a descriptive error message when using --revision-range on unsupported SCMs

2010-04-01 Thread reviewboard
Comment #9 on issue 1495 by 12gaugemedia: Need a descriptive error message when using --revision-range on unsupported SCMs http://code.google.com/p/reviewboard/issues/detail?id=1495 I'd also like to note that the documentation should also include that multiple files may be appended to the

Re: Issue 1582 in reviewboard: Dashboard Columns Misaligned

2010-04-01 Thread reviewboard
Comment #3 on issue 1582 by chipx86: Dashboard Columns Misaligned http://code.google.com/p/reviewboard/issues/detail?id=1582 Okay, I made another commit to djblets (2be2433) that should fix stale render caches, which I suspect is causing this problem. I do want to note that I couldn't

Re: Issue 1419 in reviewboard: Error uploading diff: The file was not found in the repository (207)

2010-04-01 Thread reviewboard
Updates: Status: NeedInfo Comment #9 on issue 1419 by chipx86: Error uploading diff: The file was not found in the repository (207) http://code.google.com/p/reviewboard/issues/detail?id=1419 Is the very tip of the repository http://svn02/repos/perl/, or

Issue 1584 in reviewboard: Expand (source file) is not working in 1.5beta1

2010-04-01 Thread reviewboard
Status: New Owner: Labels: Type-Defect Priority-Medium New issue 1584 by mate.szalay: Expand (source file) is not working in 1.5beta1 http://code.google.com/p/reviewboard/issues/detail?id=1584 What version are you running? 1.5beta1 What's the URL of the page containing the problem?

Re: Issue 1582 in reviewboard: Dashboard Columns Misaligned

2010-04-01 Thread reviewboard
Comment #4 on issue 1582 by tbrezins: Dashboard Columns Misaligned http://code.google.com/p/reviewboard/issues/detail?id=1582 I will test the nightly first thing tomorrow morning. Thanks. -- You received this message because you are listed in the owner or CC fields of this issue, or because

Re: Issue 1333 in reviewboard: Link to bug tracker inside description field not discovered if bug # is not a numeric value

2010-04-01 Thread reviewboard
Comment #4 on issue 1333 by lonico: Link to bug tracker inside description field not discovered if bug # is not a numeric value http://code.google.com/p/reviewboard/issues/detail?id=1333 Based on the current pattern, you would need to write bug #CATALOG-8 to get a link. Note that being

Issue 1587 in reviewboard: Some lines don't word-wrap correctly on IE/Chrome

2010-04-01 Thread reviewboard
Status: New Owner: Labels: Type-Defect Priority-Medium New issue 1587 by seagle0128: Some lines don't word-wrap correctly on IE/Chrome http://code.google.com/p/reviewboard/issues/detail?id=1587 What version are you running? ReviewBoard 1.0.6 RBTools 0.2beta2 What's the URL of the page