<div dir="ltr"><div>+1 for Nick's comment</div><div><br></div><div> Anshil<br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Oct 5, 2021 at 2:19 PM Nick Desaulniers 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">On Tue, Oct 5, 2021 at 9:05 AM Tom Stellard via cfe-dev<br>
<<a href="mailto:cfe-dev@lists.llvm.org" target="_blank">cfe-dev@lists.llvm.org</a>> wrote:<br>
><br>
> Hi,<br>
><br>
> # Proposal<br>
><br>
> The LLVM Foundation Board of Directors is seeking comment on the current state of Code Review<br>
> within the LLVM Project and its sub-projects.   Phabricator is no longer actively maintained<br>
> and we would like to move away from a self-hosted solution, so our goal is to determine if<br>
> GitHub Pull Requests are a good alternative to our current code review tool: Phabricator.<br>
><br>
> Specifically we are looking for feedback on:<br>
> - What features or properties make Github Pull Requests better than Phabricator?<br>
<br>
I appreciate the ability to have bots help participate in code review,<br>
though perhaps this isn't actually specific to github.<br>
<br>
In particular, I think rustc's use of bors for merging is pretty<br>
awesome; people literally cannot merge unless the bot has run all unit<br>
tests on all platforms.  As is, anyone can commit without running any<br>
tests.<br>
<br>
I frequently break the build for non-linux platforms. That LLVM has so<br>
many reverts for breakage on platforms tested post submit is<br>
embarrassing.  People chasing such breakages is a waste of their time,<br>
IMO.  We should have more pre-submit testing; I think the board should<br>
focus on that particular problem first, BEFORE pursuing changing code<br>
review platforms.<br>
<br>
It seems an obvious risk to me that phab is no longer maintained, but<br>
as others have noted, it hasn't all come crashing down yet.<br>
<br>
> - What features or properties  make Phabricator better than GitHub Pull Requests?<br>
> - What new workflows or process improvements will be possible with GitHub Pull Requests?<br>
> - Which workflows aren’t possible with GitHub Pull Requests?<br>
> - Any other information that you think will help the Board of Directors make the best decision.<br>
><br>
> # Where to Direct Feedback<br>
><br>
> Please provide feedback on the Infrastructure Working Group ticket[1].  This will make<br>
> it easier to collect and consolidate the responses.   At the end of the comment period<br>
> the Infrastructure Working Group will collect the feedback for further analysis and summarization.<br>
><br>
> # Timeline<br>
><br>
> The timeline for this RFC will be as follows:<br>
><br>
> - RFC posted on llvm-dev for public review and comment<br>
> - 30 days after the date of posting, public comment closes.<br>
> - IWG will have 14 days from closure of public comments to review and summarize public<br>
>    comments into a pros and cons list to be present to LLVM Foundation Board<br>
> - Foundation Board will have 30 days to make a final decision about using GitHub Pull Requests<br>
>    and then communicate a migration plan to the community.<br>
><br>
> Thank you,<br>
> LLVM Foundation Board of Directors<br>
><br>
> [1] <a href="https://github.com/llvm/llvm-iwg/issues/73" rel="noreferrer" target="_blank">https://github.com/llvm/llvm-iwg/issues/73</a><br>
><br>
> _______________________________________________<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>
<br>
<br>
<br>
-- <br>
Thanks,<br>
~Nick Desaulniers<br>
_______________________________________________<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>