<div dir="ltr">On Tue, Dec 1, 2015 at 1:00 PM, Michael Comella <span dir="ltr"><<a href="mailto:michael.l.comella@gmail.com" target="_blank">michael.l.comella@gmail.com</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><span class="">On Sat, Nov 28, 2015 at 7:56 AM, Mark Finkle <span dir="ltr"><<a href="mailto:mfinkle@mozilla.com" target="_blank">mfinkle@mozilla.com</a>></span> wrote:<br></span><div class="gmail_extra"><div class="gmail_quote"><span class=""><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">When I run | mach eslint | I get this error:<div>







<p><span>Error: Cannot find module 'eslint-plugin-mozilla'</span></p><p><span>How do I get that plugin?</span></p></div></div></blockquote></span><div>I filed a bug to either alert the user to warn this if they haven't done so already, or to just run it automatically:<br>  <a href="https://bugzilla.mozilla.org/show_bug.cgi?id=1211483" target="_blank">https://bugzilla.mozilla.org/show_bug.cgi?id=1211483</a><br><br></div><div>Given the sudo issues another person posted, we should warn about that too. It'd be great to get some traction on this – it's a confusing experience!<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><div><div class="gmail_extra"><br><div class="gmail_quote"><span class="">On Fri, Nov 27, 2015 at 5:53 PM, Dave Townsend <span dir="ltr"><<a href="mailto:dtownsend@mozilla.com" target="_blank">dtownsend@mozilla.com</a>></span> wrote:<br></span><span class=""><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
We don't really have automation testing this right now but hopefully<br>
we will soon. For now you can add support to editors like Sublime to<br>
tell you when you're making mistakes:<br>
<a href="https://blog.mozilla.org/standard8/2015/05/13/using-eslint-alongside-the-firefox-hello-code-base-to-help-productivity/" rel="noreferrer" target="_blank">https://blog.mozilla.org/standard8/2015/05/13/using-eslint-alongside-the-firefox-hello-code-base-to-help-productivity/</a><br></blockquote></span></div></div></div></div></blockquote><div><br></div><div>We've had eslint working in mobile for a while but frankly, I don't think anyone has used it (e.g. we have failures now). I don't think this will become truly useful without some automated testing – is there a bug open for this?<br></div></div></div></div></blockquote><div><br></div><div>I don't know if there's a bug but this type of work is on the list for MozReview and it should be integrated other places as well. The key is to catch regressions like these before they are committed to the tree.<br><br></div><div>Lawrence<br> <br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div><br></div><div>I started some of this work in <a href="https://bugzilla.mozilla.org/show_bug.cgi?id=1180343" target="_blank">https://bugzilla.mozilla.org/show_bug.cgi?id=1180343</a> and got it working via the web interface – should we requisition that bug?<br></div><div>- Mike (:mcomella)<br></div></div></div></div>
<br>_______________________________________________<br>
firefox-dev mailing list<br>
<a href="mailto:firefox-dev@mozilla.org">firefox-dev@mozilla.org</a><br>
<a href="https://mail.mozilla.org/listinfo/firefox-dev" rel="noreferrer" target="_blank">https://mail.mozilla.org/listinfo/firefox-dev</a><br>
<br></blockquote></div><br></div></div>