1881262 - Explicitly turn off ESLint rule no-useless-escape across the tree

Status: RESOLVED FIXED

(Developer Infrastructure :: Lint and Formatting, task, P3)

Description

[Mark Banner (:standard8)]

The ESLint rule no-useless-escape is currently turned off in our recommended configuration, but turned on for security/ and taskcluster/docker/periodic-updates/.

:mossop, :Gijs and myself met and discussed potentially enabling this rule across the whole tree, however we decided that it doesn't catch any real errors (only handles simplifications), and the work to enable it would be difficult without much benefit.

Therefore we've decided to not enable it. As we want to try and get consistent rules across the tree as far as possible and make our overall configuration as simple, I think we should also disable it in the extra two directories as well.

Comment 1

[Mark Banner (:standard8)]

Attachment: Bug 1881262 - Disable ESLint rule no-useless-escape across the code as it isn't very useful. r?Mossop!

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/1d481880f1b4
Disable ESLint rule no-useless-escape across the code as it isn't very useful. r=mossop

Comment 3

[pstanciu]

https://hg.mozilla.org/mozilla-central/rev/1d481880f1b4