<div dir="ltr">+1<div><br></div><div>I expect there will be updates and improvements to GH reviews over time. e.g. the 'show notes' checkbox to hide the notes when looking at the changes. I don't recall seeing that last week. I think gitlab's funding and cadence if putting pressure on GH to improve their UI and feature set. </div><div><br></div><div>They are just tools. Neither is perfect, but I find GH to be less problematic and less bumpy in my general work flow. I've lost work in RB more than a few times, which is not an experience I've had with GH. I find the diffs in RB inscrutable at times and have had go to look a them in GH. </div><div><br></div><div>I'd be happy to try gerrit. I haven't used it, but everyone I've spoken with who has reported a positive experience. I'd also be happy to try gitlab, but that is another, enormous, can of worms.</div><div><br></div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Oct 13, 2016 at 3:59 PM, Ian Booth <span dir="ltr"><<a href="mailto:ian.booth@canonical.com" target="_blank">ian.booth@canonical.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">-<wbr>100000000000000000000000000000<wbr>00000000000000 :-)<br>
<span class=""><br>
On 14/10/16 08:44, Menno Smits wrote:<br>
> We've been trialling Github Reviews for some time now and it's time to<br>
> decide whether we stick with it or go back to Reviewboard.<br>
><br>
> We're going to have a vote. If you have an opinion on the issue please<br>
> reply to this email with a +1, 0 or -1, optionally followed by any further<br>
> thoughts.<br>
><br>
</span>>    - +1 means you prefer Github Reviews<br>
>    - -1 means you prefer Reviewboard<br>
>    - 0 means you don't mind.<br>
<span class="">><br>
> If you don't mind which review system we use there's no need to reply<br>
> unless you want to voice some opinions.<br>
><br>
</span>> The voting period starts *now* and ends my* EOD next Friday (October 21)*.<br>
<span class="">><br>
> As a refresher, here are the concerns raised for each option.<br>
><br>
</span>> *Github Reviews*<br>
><br>
>    - Comments disrupt the flow of the code and can't be minimised,<br>
>    hindering readability.<br>
>    - Comments can't be marked as done making it hard to see what's still to<br>
>    be taken care of.<br>
>    - There's no way to distinguish between a problem and a comment.<br>
>    - There's no summary of issues raised. You need to scroll through the<br>
>    often busy discussion page.<br>
>    - There's no indication of which PRs have been reviewed from the pull<br>
<span class="">>    request index page nor is it possible to see which PRs have been approved<br>
>    or otherwise.<br>
</span>>    - It's hard to see when a review has been updated.<br>
><br>
> *Reviewboard*<br>
><br>
>    - Another piece of infrastructure for us to maintain<br>
>    - Higher barrier to entry for newcomers and outside contributors<br>
>    - Occasionally misses Github pull requests (likely a problem with our<br>
>    integration so is fixable)<br>
>    - Poor handling of deleted and renamed files<br>
>    - Falls over with very large diffs<br>
>    - 1990's looks :)<br>
>    - May make future integration of tools which work with Github into our<br>
<div class="HOEnZb"><div class="h5">>    process more difficult (e.g. static analysis or automated review tools)<br>
><br>
> There has been talk of evaluating other review tools such as Gerrit and<br>
> that may still happen. For now, let's decide between the two options we<br>
> have recent experience with.<br>
><br>
> - Menno<br>
><br>
><br>
><br>
<br>
</div></div><span class="HOEnZb"><font color="#888888">--<br>
Juju-dev mailing list<br>
<a href="mailto:Juju-dev@lists.ubuntu.com">Juju-dev@lists.ubuntu.com</a><br>
Modify settings or unsubscribe at: <a href="https://lists.ubuntu.com/mailman/listinfo/juju-dev" rel="noreferrer" target="_blank">https://lists.ubuntu.com/<wbr>mailman/listinfo/juju-dev</a><br>
</font></span></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div><div dir="ltr">Reed O'Brien <div><div><span style="font-size:12.8px">✉ </span><a href="mailto:reed.obrien@canonical.com" target="_blank">reed.obrien@canonical.com</a></div><div style="font-size:12.8px">✆ <span title="Call with Google Voice">415-562-6797</span></div></div><div style="font-size:12.8px"><span title="Call with Google Voice"><br></span></div></div></div></div></div>
</div>