Closed Bug 1960715 Opened 9 days ago Closed 9 days ago

Add type checking to Content Analysis JS files

Categories

(Firefox :: Data Loss Prevention, task)

task

Tracking

()

RESOLVED FIXED
139 Branch
Tracking Status
firefox139 --- fixed

People

(Reporter: gstoll, Assigned: gstoll)

References

Details

Attachments

(1 file)

:zombie showed me how to enable this. Note that mach lint will not check this as far as I can tell, but VS Code will flag errors, which is much better than the type checking we have now (none).

This will enable VS Code to do type-checking on these files. Most of
the problems it found were fairly trivial (or wrong type declarations),
but the extra call to _removeSlowCAMessage() seems like a real problem
(will flag in the review)

Pushed by gstoll@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/8976ffea9dc0 add type-checking to Content Analysis JS files r=zombie,dlp-reviewers,handyman
Status: ASSIGNED → RESOLVED
Closed: 9 days ago
Resolution: --- → FIXED
Target Milestone: --- → 139 Branch
Duplicate of this bug: 1960690
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: