<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Aug 2, 2016 at 4:24 PM, R. David Murray <span dir="ltr"><<a href="mailto:rdmurray@bitdance.com" target="_blank">rdmurray@bitdance.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Sat, 30 Jul 2016 23:21:07 +0200, Maciej Szulik <<a href="mailto:soltysh@gmail.com">soltysh@gmail.com</a>> wrote:<br>
> I'm leaning towards just adding an information who left the comment<br>
> and a link to the PR. I agree with Senthil that gh comments,<br>
> especially those coming from reviews have code context, which will get<br>
> lost when copying over. Besides the amount does not matter in that<br>
> case, whoever is interested in looking or answering into the patch<br>
> will have to go to GitHub and see what exactly it's about. I'm aware<br>
> there are cases you just want to read the comment and don't do<br>
> anything yet, but these are rare cases we can initially ignore. Let's<br>
> start simple and we can always get back to this topic.<br>
<br>
</span>If github comment threading were more sensible I think I'd prefer to see<br>
the comments reflected.  But since it *isn't* (it is pretty much useless<br>
outside of the web UI, and even in the web UI it is often awkward),<br>
I think linking to the PR is indeed probably better.<br>
<br>
Just to confirm, we are talking about a new link summarizing the comment<br>
activity for the past N minutes, whenever commenting activity happens,<br>
right?  It would be nice to link directly to the new comments, but somehow<br>
I doubt that is going to be possible (at least if we batch them), so we'll<br>
probably have to settle for just linking the summary to the PR as a whole.<br>
<span class="HOEnZb"></span><br></blockquote></div><br></div><div class="gmail_extra">Correct. The links to separate comments might be of no use after a rebase<br></div><div class="gmail_extra">to a PR, since they will point to hidden comments, that's why having a single<br></div><div class="gmail_extra">global link is much better, because going to the PR will give you the current<br></div><div class="gmail_extra">state of it.<br><br></div><div class="gmail_extra">Maciej<br></div></div>