<div dir="ltr"><div>Hi Martin,</div><div dir="ltr"><br></div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">I don't mind formalizing that reviews are done on phabricator only. <br>
However following projects that way would, most probably, have one <br>
quite notable drawback compared with the current mailing list based <br>
approach:<br>
<br>
Right now, it's easy to distinguish between mails requiring different <br>
levels of attention; ones with me in the To or CC fields are more visible <br>
and I try to read all of them. I have personal Herald rules that CC me on <br>
topics that I track. But I also browse the rest of the mails (quickly <br>
glancing usually only) for other topics I might be interested in.<br></blockquote><div><br></div><div>Having your own, custom Herald rules is always superior to general rules for a project. They are naturally targeted towards your use cases. However I wanted to offer a proper email integration for all users without having to write their own rules. So the idea was to offer a "similar enough" alternative for the XXX-commits mailing lists. </div><div><br></div><div>I just checked your rules [1] and you add yourself to the list of subscribers for certain revisions. For these notifications you should be on the "TO" section of the email, right?</div><div><br></div><div> The emails going through the project [1] are sent as CC to me. There is a ton of header attributes that could be used for filtering:</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"><font face="monospace">X-Phabricator-Cc: <PHID-PROJ-6nrw7h47scgenrj2njpx></font> </blockquote><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><font face="monospace">X-Herald-Rules: <74>, <368>, <665>, <667>, <671>, 700>, <576>, <615>, <770></font> </blockquote><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><font face="monospace">X-Phabricator-Stamps: actor(@bruno) application(Differential) author(@bruno) herald(H74) herald(H368) herald(H576) herald(H615) herald(H665) herald(H667) herald(H671) herald(H700) herald(H770) monogram(D99434) object-type(DREV) phid(PHID-DREV-6ivftbt7xso57bvmy2br) reviewer(@aralisza) reviewer(@delcypher) reviewer(@dvyukov) reviewer(@kubamracek) reviewer(@vitalybuka) reviewer(@yln) revision-status(needs-review) subscriber(@hoy) subscriber(@jfb) subscriber(@kubamracek) subscriber(@llvm-commits) subscriber(@lxfind) subscriber(@modimo) subscriber(@rjmccall) subscriber(@t.p.northover) subscriber(@wenlei) tag(#llvm) via(web)</font></blockquote><div><br></div><div>Do you think this is good enough for filtering?</div><div><br></div><div>[1] <a href="https://reviews.llvm.org/H746">https://reviews.llvm.org/H746</a></div><div>[2] <a href="https://reviews.llvm.org/H770">https://reviews.llvm.org/H770</a> </div><div><br></div><div><br></div><div>Best,</div><div>Christian</div></div></div>