<div dir="ltr">@Renato: What's your take on Ed's idea?<div><br></div><div>We use phab for all sorts of reviews, but it should be possible to figure out which repository a review is intended to land on and add cfe-commits or llvm-commits appropriately.</div><div><br></div><div>Although, for throw-away reviews, it might generate too much spam.</div><div><br></div><div>Cheers,</div><div><br></div><div>/ Asiri</div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Mar 8, 2017 at 3:18 PM, Ed Schouten <span dir="ltr"><<a href="mailto:ed@nuxi.nl" target="_blank">ed@nuxi.nl</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Asiri,<br>
<span class=""><br>
2017-03-07 20:42 GMT+01:00 Asiri Rathnayake <<a href="mailto:asiri.rathnayake@gmail.com">asiri.rathnayake@gmail.com</a>>:<br>
> Could you please always include cfe-commits as a subscriber in you phab<br>
> reviews?<br>
><br>
> We would like to be aware of these changes in advance before they land.<br>
<br>
</span>Sure thing! I'll try to do that from now on.<br>
<br>
That said, if the policy is to add cfe-commits@ to all<br>
Clang/libunwind-related code reviews, would it make sense to configure<br>
Phabricator's Herald to set this up for us automatically? Looking at<br>
<a href="https://reviews.llvm.org/herald/new/" rel="noreferrer" target="_blank">https://reviews.llvm.org/<wbr>herald/new/</a>, I suspect that can only be<br>
configured with admin rights.<br>
<br>
Regards,<br>
<span class="HOEnZb"><font color="#888888">--<br>
Ed Schouten <<a href="mailto:ed@nuxi.nl">ed@nuxi.nl</a>><br>
Nuxi, 's-Hertogenbosch, the Netherlands<br>
KvK-nr.: 62051717<br>
</font></span></blockquote></div><br></div>