Closed Bug 1284438 Opened 8 years ago Closed 8 years ago

Enable more ESLint rules for PSM

Categories

(Core :: Security: PSM, defect, P1)

Product:
Core
Component:
Security: PSM
Type:
defect

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla50
Tracking Flags:
Tracking Status
firefox50 --- fixed

People

(Reporter: Cykesiopka, Assigned: Cykesiopka)

References

Details

(Whiteboard: [psm-assigned])

Attachments

(1 file)

Bug 1284438 - Enable more ESLint rules for PSM.
(deleted), text/x-review-board-request
mgoodwin
: review+
Details
There are some more ESLint rules that can be enabled for PSM that should help catch bugs.
PSM JS code already pass these rules, so enabling these rules will just help catch future bugs. Review commit: https://reviewboard.mozilla.org/r/62416/diff/#index_header See other reviews: https://reviewboard.mozilla.org/r/62416/
Attachment #8768107 - Flags: review?(mgoodwin)
Comment on attachment 8768107 [details] Bug 1284438 - Enable more ESLint rules for PSM. https://reviewboard.mozilla.org/r/62416/#review60014
Attachment #8768107 - Flags: review?(mgoodwin) → review+
Pushed by cbook@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/df769eb7552e Enable more ESLint rules for PSM. r=mgoodwin
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/df769eb7552e
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
status-firefox50: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla50
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: