1851794 - Change debugger ESLint rules that are warnings to errors, and turn off react/prop-types.

Status: RESOLVED FIXED

(DevTools :: Debugger, task)

Description

[Mark Banner (:standard8)]

I'm looking to reduce the cases where we have long term ESLint rules reporting as warnings. If we have reason to be raising an issue, then we should ensure it is addressed, rather than letting it languish in amongst other warnings where it can become ambiguous as to why it hasn't been addressed.

In debugger there are a number of rules that are currently marked as warnings, even though they don't have any issues raised currently. We should transition those to errors.

For the react/prop-types rule, there are issues raised, but the plan in bug 1841645 is to remove propTypes definitions from the debugger code base, hence it will no longer be needed so it can be turned off here.

Comment 1

[Mark Banner (:standard8)]

Attachment: Bug 1851794 - Change debugger ESLint rules that are warnings to errors, and turn off react/prop-types. r?#devtools-reviewers!

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/19d1224f216a
Change debugger ESLint rules that are warnings to errors, and turn off react/prop-types. r=devtools-reviewers,ochameau

Comment 3

[Cristina Horotan [:chorotan]]

https://hg.mozilla.org/mozilla-central/rev/19d1224f216a