Rename mailServices.js to MailServices.jsm

RESOLVED FIXED in Thunderbird 63.0

Status

enhancement
RESOLVED FIXED
Last year
Last year

People

(Reporter: darktrojan, Assigned: darktrojan)

Tracking

unspecified
Thunderbird 63.0
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

I'm tired of dealing with the fact this file has a lowercase m, and why isn't it a .jsm? I'm going to fix it, and leave a shim in place because there's bound to be extensions expecting it.
Blocks: 1485886
Attachment #9005522 - Flags: review?(mkmelin+mozilla)
Comment on attachment 9005522 [details] [diff] [review]
1487572-rename-mailservices-1.diff

Review of attachment 9005522 [details] [diff] [review]:
-----------------------------------------------------------------

Makes sense to me. r=mkmelin
Attachment #9005522 - Flags: review?(mkmelin+mozilla) → review+
Ship it, quick before the bitrot takes hold!
Keywords: checkin-needed
Pushed by mozilla@jorgk.com:
https://hg.mozilla.org/comm-central/rev/aa9abd3271f5
Rename mailServices.js to MailServices.jsm. r=mkmelin
Status: ASSIGNED → RESOLVED
Closed: Last year
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 63.0
Finally :)
Surely eslint will be happier too :)
Blocks: 819770
You need to log in before you can comment on or make changes to this bug.