Closed Bug 1881262 Opened 5 months ago Closed 5 months ago

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

Categories

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

Tracking

(firefox125 fixed)

RESOLVED FIXED
Tracking Status
firefox125 --- fixed

People

(Reporter: standard8, Assigned: standard8)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

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.

Pushed by mbanner@mozilla.com:
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
Status: NEW → RESOLVED
Closed: 5 months ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: