<div dir="ltr"><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Jan 14, 2020 at 11:32 AM Renato Golin via llvm-dev <<a href="mailto:llvm-dev@lists.llvm.org">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">On Wed, 8 Jan 2020 at 02:26, Daniel Sanders via llvm-dev<br>
<<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a>> wrote:<br>
> It's worth mentioning that Phabricator can read strings of the format 'Depends on D1234' from commit messages and create those relationships for you.<br>
<br>
Side note: I find that there's too many unknown features of Phab that<br>
require manual annotations on the commit message to work.<br>
<br>
I don't think that's a good strategy. New contributors get lost in the<br>
specifics and reviewers forget to do them.<br>
<br>
> Also just because it's not easy to find unless you know it's there. You can view the parent/child relationships in the 'Revision Contents' section on the 'Stack' tab.<br>
<br>
The parental relationship in Phab is not obvious. I can't easily see<br>
it in the snapshot and often ask people to link the commits in order<br>
when they already are.<br>
<br>
I'd also like to see the whole series without having to navigate the<br>
linked list.<br>
<br>
Git allows multiple commits per pull request, so does GitHub's PR UI,<br>
as well as showing all the other changes (force push, rebase, reorder,<br>
additional fixups), which makes it much simpler to see what changed.<br></blockquote><div>Except the GitHub individual commit views are rather terrible for adding comments to. The individual commits are not treated as separable commits for approval purposes, and GitHub decides on its own that freshly added comments are outdated due to surrounding noise.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
Phab is better at keeping track of old comments, but where GitHub<br>
completely looses the comment (on history change),</blockquote><div>That aspect of GitHub makes longer running reviews quickly unusable. It pretty much only works if all reviewers ensure that the PR originator addresses all comments before they "scroll off".<br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Phab moves the<br>
comment to a random place in the file, which is equally broken.<br></blockquote><div>Much less broken IMO.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
Granted, GitHub's UI is much "simpler" than Phab, but to my view, this<br>
is not a problem, but a benefit.<br></blockquote><div>GitHub's UI is a screen real-estate hog due to low information density even when collapsing things that you rather should see.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
If we moved to GitHub PRs today, I wouldn't miss a thing.<br>
<br>
cheers,<br>
--renato<br>
_______________________________________________<br>
LLVM Developers mailing list<br>
<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a><br>
<a href="https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev" rel="noreferrer" target="_blank">https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev</a><br>
</blockquote></div></div>