<div dir="ltr"><div dir="ltr">Hi,<div><br></div><div>This was part of what I proposed when we integrated MLIR in LLVM ( <a href="http://lists.llvm.org/pipermail/llvm-dev/2019-November/136579.html">http://lists.llvm.org/pipermail/llvm-dev/2019-November/136579.html</a> ), we were already using pull-requests before and had CI infrastructure already built for it. </div><div>Ultimately we didn't press this further, divergence inside the repo / between the subproject does not seems desirable in my opinion: </div><div>- it creates confusion: "why is this repository having pull-requests and reviews on GitHub but my pull-request gets automatically closed?", "I followed doc X"</div><div>- the lack of Herald on GitHub makes it so that we can't filter / subscribe automatically to individual pull-requests: this was the major blocker in my opinion, I couldn't find a solution to this and I believe it is critical.</div><div>- it does not favor to build common tooling: the recent work on enabling pre-submit CI tests on Phabricator is valuable and I'm looking forward to get this extended. But splitting the various ways of contributing to the repo just means more infrastructure to build to sustain this kind of efforts. (the infrastructure is easier built on GitHub by the way, but that is an argument in favor of migrating from Phab to GH for the full-project).</div><div><br></div><div>So in summary: I'd rather find a path for the full-project to do this, but acknowledging that there are few blockers to solve before getting there (cf other threads on the topic).</div><div><br></div><div>-- </div><div>Mehdi</div></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Feb 20, 2020 at 10:21 AM Louis Dionne 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">Hi,<br>
<br>
I know there has been significant discussion about "moving" from Phabricator to GitHub reviews and pull requests, etc. I'm not suggesting that we do anything in terms of global LLVM policy. However, as a maintainer of libc++, I commit __a lot__ of other people's code for them. It would be a huge time saver for me if I could nicely suggest to contributors (not force them) to use PRs instead of Phabricator for their contributions. It would also handle commit attribution properly, which is a pain right now.<br>
<br>
Would it be possible to allow GitHub PRs to be submitted on the monorepo so as to let individual sub-projects deal with it however they please? I've spoken to numerous people involved in libc++ development and they would like to start submitting PRs (and for the others, we'll still accept Phabricator reviews). Perhaps it is possible to setup some kind of filter such that PRs touching only libcxx/ and libcxxabi/ can be submitted, but otherwise they're closed by the bot?<br>
<br>
Cheers,<br>
Louis<br>
<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>