1772358 - Create an ESLint rule to reject defining lazy getter for Services etc

Status: RESOLVED FIXED

(Developer Infrastructure :: Lint and Formatting, task)

Description

[Tooru Fujisawa [:arai]]

Some system modules (JSM) are loaded in early startup and not necessarily be lazy getter.
bug 1608279 and depending patches discovered that there are too many cases that those modules are defined as lazy getter.
we should have an ESLint rule that rejects such definition.

Comment 1

[Tooru Fujisawa [:arai]]

This can be implemented by extending bug 1773747 rule, to always mark Services etc eager.

Comment 2

[Tooru Fujisawa [:arai]]

bug 1773747 is about all lazy getters, not only about import, so the logic is slightly different, and adding separate rule is better.

Comment 3

[Tooru Fujisawa [:arai]]

Attachment: Bug 1772358 - Add ESLint rule to reject defining lazy getters for always available modules. r?Standard8!

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/9779ccacc2d7
Add ESLint rule to reject defining lazy getters for always available modules. r=Standard8

Comment 5

[Cristina Cozmuta (:CrissCozmuta)]

https://hg.mozilla.org/mozilla-central/rev/9779ccacc2d7