<div dir="ltr"><div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Jan 14, 2020 at 10:46 AM David Greene 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-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex">Renato Golin via llvm-dev <<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a>> writes:<br>
<br>
> Honestly, the fact that I have to either user Arcanist or edit the<br>
> comments metadata by hand or use the UI to do a simple task is asking<br>
> a bit too much.<br>
<br>
This brings something else to mind for me.  Another advantage of GitHub<br>
is that there are many tools out there that work with it.  Not so much<br>
for Phab.  As an Emacs user, getting access to magit/forge's ability to<br>
interact with GitHub PRs would be amazing.<br></blockquote><div><br></div><div>This is a reason I proposed using PR when we were looking to merge MLIR: GitHub has an ecosystem of third-party tooling, for example we were relying on this service: <a href="https://codecov.io/gh/tensorflow/mlir">https://codecov.io/gh/tensorflow/mlir</a> which provide incremental code coverage annotation in GitHub pull-request.</div><div>That said there has been investment in pre-merge testing on our Phabricator instance in the meantime, so I'm fairly happy about the improvement there right now.<br></div><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex">
<br>
> I think we need to move from "how apt is GitHub's PR UI in emulating<br>
> what Phab does" to "what do we really need from a review UI and how<br>
> simple it is the process for both contributors and reviewers".<br>
<br>
Absolutely.  Reviewing a patch series has always seemed a bit counter to<br>
LLVM's development philosophy of small, incremental changes.  I<br>
understand the need to provide context for large changes (I have posted<br>
mega-patches in Phab for that purpose), but the path to get to the large<br>
change has always been small, incremental changes.<br></blockquote><div><br></div><div>There is a counter point to this: when folks doing a change needs a refactoring, they are more likely to submit both in a single review in a workflow that does not support patch series.</div><div><br></div><div>Even though I proposed to use GitHub PR a couple of months ago, the code review experience seems not as good as Phabricator to me on some aspects (on some other aspects, GitHub has been improving beyond our version of Phabricator though, like proposing inline changes when reviewing).</div><div><br></div><div>Best,</div><div></div><div><br></div><div>-- </div><div>Mehdi</div><div> </div></div></div></div>