<div dir="ltr">+1<br><div><br></div><div>Keeping the PR and reviews together really makes it easier for me to keep track of what's going on with a PR.  It's also really nice not having to context switch out of github for every single PR.</div><div><br></div><div>Reviewboard and related infrastructure breaks like once couple weeks, and I'm not convinced it'll get better, since we've been using it for quite some time now.</div><div><br></div><div>I have missed exactly zero of the features of reviewboard since using github, and haven't really cared about the drawbacks of github.</div><div><br></div><div>One point - you *can* minimize comments in the files view - there's a checkbox per file that will hide the comments in that file.</div></div><br><div class="gmail_quote"><div dir="ltr">On Fri, Oct 14, 2016 at 8:22 AM roger peppe <<a href="mailto:rogpeppe@gmail.com">rogpeppe@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 14 October 2016 at 12:45, Adam Collard <<a href="mailto:adam.collard@canonical.com" class="gmail_msg" target="_blank">adam.collard@canonical.com</a>> wrote:<br class="gmail_msg">
> Not sure I get a vote, but -1<br class="gmail_msg">
><br class="gmail_msg">
> You're running an old version of ReviewBoard (2.0.12 released in January<br class="gmail_msg">
> 2015) and many of the issues I think you've been hitting are fixed in later<br class="gmail_msg">
> revisions. Latest stable is 2.5.6.1, 3.0.x is under active development and<br class="gmail_msg">
> brings a chunk of new UI improvements.<br class="gmail_msg">
><br class="gmail_msg">
> Release notes for 2.5<br class="gmail_msg">
><br class="gmail_msg">
> 3.0 demo site<br class="gmail_msg">
<br class="gmail_msg">
I'm still not convinced.<br class="gmail_msg">
<br class="gmail_msg">
Even 3.0 still deletes draft comments without so much as a by-your-leave<br class="gmail_msg">
when you double-click somewhere else in the text. And because it doesn't use<br class="gmail_msg">
real text entry boxes, the Lazarus plugin, my usual saviour in such cases,<br class="gmail_msg">
doesn't work. I've lost far too much time to this in the past.<br class="gmail_msg">
<br class="gmail_msg">
Replying to a comment still involves a page reload and associated lost context.<br class="gmail_msg">
<br class="gmail_msg">
I can't see anything in the 2.5 release notes about fixing behaviour on file<br class="gmail_msg">
move/rename, though I may well have missed it.<br class="gmail_msg">
<br class="gmail_msg">
And not being able to deal with really large PRs is a definite issue too (not<br class="gmail_msg">
that github is better there).<br class="gmail_msg">
<br class="gmail_msg">
  cheers,<br class="gmail_msg">
    rog.<br class="gmail_msg">
<br class="gmail_msg">
--<br class="gmail_msg">
Juju-dev mailing list<br class="gmail_msg">
<a href="mailto:Juju-dev@lists.ubuntu.com" class="gmail_msg" target="_blank">Juju-dev@lists.ubuntu.com</a><br class="gmail_msg">
Modify settings or unsubscribe at: <a href="https://lists.ubuntu.com/mailman/listinfo/juju-dev" rel="noreferrer" class="gmail_msg" target="_blank">https://lists.ubuntu.com/mailman/listinfo/juju-dev</a><br class="gmail_msg">
</blockquote></div>