<div dir="ltr"><div dir="ltr"></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Jun 25, 2020 at 11:24 AM Johannes Doerfert 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">
  
    
  
  <div>
    <p><br>
    </p>
    <div>On 6/24/20 3:41 PM, Chris Lattner via
      llvm-dev wrote:<br>
    </div>
    <blockquote type="cite">
      <blockquote type="cite">
        <pre>When I looked into this possibility (GitHub PR) when we merged MLIR in the monorepo: just Herald rules was already a blocker and I couldn't find a replacement readily available on GitHub. The handling of stack of revisions on GitHub was another problem mentioned.
</pre>
      </blockquote>
      <pre>Fair enough - I haven’t investigated the technical issues with adopting GitHub PRs for LLVM.  My experience with the Swift community has been extremely positive though, I’d be surprised if there is something particularly crazy that only LLVM needs that GitHub hasn’t encountered.  That said, again, I don’t have any practical experience with this part of the stack,</pre>
    </blockquote>
    <p>I am very much relying on Herald rules to filter patches (=PRs).</p>
    <p> We need some replacement, IMHO, as we have many more
      patches/commits than swift and the mono-repo is more diverse (I
      would imagine).</p></div></blockquote><div><br></div><div>Yes, but this part of the problem is trivial to solve. We can add a github workflow which triggers on pull-requests being uploaded or modified, which will @ people who ask to be, based on a configuration file matching commit paths/descriptions. There may even be something existing which does this, but even if not, it should be no more than a week to fully implement.</div><div><br></div><div>If resolving this would actually let us migrate to GitHub, I'd volunteer to implement it myself, but it seems like there's other issues which are less trivial to solve that might also block it.</div><div><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><p>
    </p>
    <p>~ Johannes<br>
    </p>
    <p><br>
    </p>
    <blockquote type="cite">
      <pre>-Chris
</pre>
      <br>
      <fieldset></fieldset>
      <pre>_______________________________________________
LLVM Developers mailing list
<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a>
<a href="https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev" target="_blank">https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev</a>
</pre>
    </blockquote>
  </div>

_______________________________________________<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>