<div dir="ltr">If we're finding too many shortcomings with the GH code review system we could look at Gerrit or something else.<div><br></div><div>It looks like people have already done Gerrit/Github integration before:</div>
<div><a href="http://gerrithub.io/">http://gerrithub.io/</a><br></div><div><a href="http://www.packtpub.com/article/using-gerrit-with-github">http://www.packtpub.com/article/using-gerrit-with-github</a><br></div><div><br>
</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On 6 June 2014 12:06, Andrew Wilkins <span dir="ltr"><<a href="mailto:andrew.wilkins@canonical.com" target="_blank">andrew.wilkins@canonical.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><div class="h5"><div class="gmail_extra"><div class="gmail_quote">On Thu, Jun 5, 2014 at 9:26 PM, roger peppe <span dir="ltr"><<a href="mailto:rogpeppe@gmail.com" target="_blank">rogpeppe@gmail.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div>On 5 June 2014 14:22, Andrew Wilkins <<a href="mailto:andrew.wilkins@canonical.com" target="_blank">andrew.wilkins@canonical.com</a>> wrote:<br>


> On Thu, Jun 5, 2014 at 9:02 PM, roger peppe <<a href="mailto:rogpeppe@gmail.com" target="_blank">rogpeppe@gmail.com</a>> wrote:<br>
>><br>
>> On 5 June 2014 05:07, Menno Smits <<a href="mailto:menno.smits@canonical.com" target="_blank">menno.smits@canonical.com</a>> wrote:<br>
>> > One nice feature is that old comments are hidden<br>
>> > if the area of the code they reference is changed by a later diff.<br>
>><br>
>> I consider this a serious shortcoming. I generally want to see all the<br>
>> conversation for a given code review. Is it possible to reveal<br>
>> the old comments somehow?<br>
><br>
><br>
> See the comment history on: <a href="https://github.com/juju/juju/pull/8" target="_blank">https://github.com/juju/juju/pull/8</a><br>
><br>
> - I proposed, and jam made some comments<br>
> - I addressed the comments in a new commit, rebased and squashed it into the<br>
> old, and did a "git push -f"<br>
><br>
> There's now a "jameinel commented on an outdated diff". Click on "Show<br>
> outdated diff" to the right, and you can see the old comments.<br>
><br>
> (I was under the impression that old comments got dropped altogether after<br>
> rebase. Not sure if behaviour in GitHub changed at some point, or if it was<br>
> always like this...)<br>
<br>
</div></div>Ah, that's better than I thought, thanks.<br>
<br>
Is there a straightforward way of seeing the changes made in response<br>
to a particular I(outdated) comment? That's something I did almost every<br>
time in Rietveldt.<br>
</blockquote></div><br></div></div></div><div class="gmail_extra">Not AFAIK. I miss this too.</div></div>
</blockquote></div><br></div>