
<html>

  <head>

    <meta http-equiv="Content-Type" content="text/html; charset=UTF-8">

  </head>

  <body>

    <div class="moz-cite-prefix">On 19/09/2019 22:20, Dave Townsend

      wrote:<br>

    </div>

    <blockquote type="cite"

cite="mid:CAPMxTNrx53NXAHkb3VrGJsr9zacZseoM_8KhF3Bin8bovahb4w@mail.gmail.com">

      <meta http-equiv="content-type" content="text/html; charset=UTF-8">

      <div dir="ltr">

        <div>So apparently the XUL pre-processer we use for eslint

          doesn't support inline scripts, the errors you're seeing are

          from code that just wasn't being linted before. I think it

          would be fine for now to just add them to the ignored list but

          it would be good to put a bug on file to get them fixed up

          though.</div>

      </div>

    </blockquote>

    <p>The XUL preprocessor also disables some rules iirc. The

      preprocessing was added to get at least some linting there, but

      with xul going away we didn't want to do lots of work on it.</p>

    <p>The best way to "fix" these is probably adding an entry to the

      top-level <a moz-do-not-send="true"

        href="https://searchfox.org/mozilla-central/source/.eslintrc.js">.eslintrc.js

        file</a>, and disable just the rules that are failing for that

      directory. That way, we get the majority of rules enabled, and can

      work on the remaining ones (these tend to make good mentored

      bugs).</p>

    <p>You'll see that there's already various directory/rule

      combinations that we have disabled currently, and there's various

      bugs in place to get these enabled.<br>

    </p>

    Mark<br>

  </body>

</html>