<div dir="ltr"><div>Relatedly, Phabricator doesn't stop you continuing a comment chain for reasons I have yet to follow, which Github sometimes does.</div><div><br></div><div>Some others:<br></div><div>1) I believe Github also doesn't have an easy way to respond to multiple comments simultaneously, if you are not in "review" mode, (which is always the case if you are replying to out-of-line comments).<br></div><div>2) Typically in our Phabricator, the description and summary of a patch is considered to be roughly what goes in the final commit message (same with Github). However, updating said commit message isn't possible to my knowledge without force-pushing in Github (which as discussed elsewhere in the thread causes other problems).</div><div>3) Marking comments as "Done" in Phabricator does not auto-hide them, whereas in Github marking a comment as "Resolved" does. Spoken from experience, this leads to situations in Github where a developer thinks they've resolved a reviewers comments, marks them as Resolved, but actually they weren't. The reviewer in turn then has to browse the comments in the summary page, to see if they have any unaddressed comments that were supposedly resolved, which given the limited context available there is a non-trivial task sometimes.</div><div><br></div><div>There are probably others, but those are the ones not already mentioned that I can come up with so far.</div><div><br></div><div>P.S. Thanks Mehdi/Fangrui for stepping up! Whilst I'm not opposed to working with Github towards getting the feature parity with Phabricator sorted, I don't want to switch until the two are on a par with each other, which in my opinion is not yet, so being forced to do so prematurely due to lack of maintainers would have made me very sad (P.P.S thank you Manuel for Phabricator in the first place!).<br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, 25 Jun 2020 at 22:39, Hubert Tong via cfe-dev <<a href="mailto:cfe-dev@lists.llvm.org">cfe-dev@lists.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Jun 25, 2020 at 12:29 PM Mehdi AMINI via llvm-dev <<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div dir="ltr"><div dir="ltr"><br></div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div>Re: patch chains - perhaps we can ask github on support for that?</div><div>I think what would help is somebody providing a doc with the features that phab provides that github PR doesn't, so we can get some consensus on what the diff is.</div></div></blockquote><div><br></div><div>Indeed, I'll try to start a doc on this.<br></div></div></div></div></blockquote><div>Thanks. Some things that are high on my list:</div><div>Phabricator has persistent per-user inline comment collapsing.</div><div>
<div>Phabricator can show all inline comments over the lifetime of the patch in-line (even if the placement is not perfect) in the full diff view.</div><div><br></div>

</div></div></div>
_______________________________________________<br>
cfe-dev mailing list<br>
<a href="mailto:cfe-dev@lists.llvm.org" target="_blank">cfe-dev@lists.llvm.org</a><br>
<a href="https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-dev" rel="noreferrer" target="_blank">https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-dev</a><br>
</blockquote></div>