<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Fri, Feb 23, 2018 at 5:39 PM, Ed Lee <span dir="ltr"><<a href="mailto:edilee@mozilla.com" target="_blank">edilee@mozilla.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On Thu, Feb 1, 2018 at 10:48 PM, Ed Lee <<a href="mailto:edilee@mozilla.com">edilee@mozilla.com</a>> wrote:<br>
> Great to see these types of broad changes getting wins, so if there's<br>
> a good way to keep up to date and ahead of these types of incoming<br>
> changes, that would be great.<br>
Just trying to see if people have an answer of being able to get some<br>
advance warning. The following bug just merged to mozilla-central, and<br>
Activity Stream development is broken again:<br>
<br>
Bug 1440284 - change this.EXPORTED_SYMBOLS back to var<br>
EXPORTED_SYMBOLS in JS modules, r=mccr8.<br>
<a href="https://hg.mozilla.org/mozilla-central/rev/b599a95ce057" rel="noreferrer" target="_blank">https://hg.mozilla.org/<wbr>mozilla-central/rev/<wbr>b599a95ce057</a><br>
<br>
At least this most recent bug and the JSM import bug were both filed<br>
in Firefox :: General, so watching that component for these potential<br>
changes probably isn't ideal. Maybe watching Testing :: Lint is a<br>
better proxy as it seems like these types of changes often have<br>
accompanying eslint rule changes, but I didn't find a bug related to<br>
this EXPORTED_SYMBOLS change.<br>
<br>
Or maybe I can convince Standard8 to specially notify us. ;)<span class="gmail-HOEnZb"><font color="#888888"><br></font></span></blockquote><div><br></div><div>You can go to <a href="https://phabricator.services.mozilla.com/herald/create/">https://phabricator.services.mozilla.com/herald/create/</a> and create a rule in Phabricator that notifies you when:</div><div><br></div><div>* A particular path changes<br></div><div>* A review touches a particular path<br></div></div></div><div class="gmail_extra"><br></div><div class="gmail_extra">Of course, the latter won't be useful until Phabricator is actively being used for reviews. But post-commit notification is usable today.<br></div><div class="gmail_extra"><br></div><div class="gmail_extra">And, there isn't a good way to identify a "broad change." But if we came up with a commit message convention for identifying "broad changes," you could subscribe to that keyword. Or if we came up with a heuristic to automatically identity "broad changes," a custom tool submitting the review (read: not Arcanist) could add/tag the review as such. Or a bot could examine incoming reviews and flag them as "broad changes" accordingly. Lots of options here.<br></div></div>