<div dir="ltr">When I run | mach eslint | I get this error:<div><br></div><div>







<p><span>Error: Cannot find module 'eslint-plugin-mozilla'</span></p><p><span><br></span></p><p><span>How do I get that plugin?</span></p></div></div><div class="gmail_extra"><br><div class="gmail_quote">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><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Thanks to some speedy work by Mark Banner and help from Mike Conley,<br>
Felipe Gomes and Gijs Kruitbosch we've now landed the changes to make<br>
it possible to run "mach eslint" on any directory in the tree.<br>
<br>
The default rules for the tree are almost non-existent so this is<br>
mostly checking for syntax errors right now. It is also disabled for<br>
most directories.<br>
<br>
Going forwards we'd like to enable it in more directories and turn on<br>
more rules. You can help! If there is a directory you work in a lot<br>
try removing it from the top-level .eslintignore file and see what<br>
fails, then fix it. Mostly it will be simple fixes.<br>
<br>
For now "mach eslint" at the top level doesn't pass. Both devtools and<br>
android have some failures so we'll be wanting to get those fixed. We<br>
also know that some of our JS syntax isn't supported by ESLint. Array<br>
generators look like they might not be standardized anyway so please<br>
switch them to Array.map. For conditional catch expressions we may<br>
want to add support to eslint somehow but for now any of those cases<br>
will just have to be ignored.<br>
<br>
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>
<br>
Thanks to everyone who jumped to make this happen today!<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>
</blockquote></div><br></div>