<div dir="ltr"><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Jun 25, 2020 at 12:33 PM James Y Knight 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"><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" target="_blank">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> 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.<br>
</div></div></div><br></blockquote><div><br></div><div>It is not quite a full replacement for Herald rules, but Github supports labeling PRs depending on what files they modify:</div><div><br></div><div><a href="https://github.com/actions/labeler">https://github.com/actions/labeler</a></div><div><br></div><div>You can see it in action in the wasmtime repo, with its labels:</div><div><br></div><div><a href="https://github.com/bytecodealliance/wasmtime/blob/main/.github/labeler.yml">https://github.com/bytecodealliance/wasmtime/blob/main/.github/labeler.yml</a></div><div><br></div><div>and then using the subscribe-to-label action:</div><div><br></div><div><a href="https://github.com/bytecodealliance/subscribe-to-label-action">https://github.com/bytecodealliance/subscribe-to-label-action</a></div><div><br></div><div>people can get automatically CC'd to whatever areas they like.  The only downside is that information lives in the repo itself, so people who want to be notified have to submit "fixes" to the repo.<br></div><div><br></div><div>I know Herald lets you automatically add reviewers for given paths; it seems like that functionality ought to be supportable through Github actions as well.</div><div><br></div><div>-Nathan<br> </div></div></div>