Make browser/modules eslintable

RESOLVED FIXED in Firefox 46

Status

()

defect
RESOLVED FIXED
4 years ago
3 years ago

People

(Reporter: Gijs, Assigned: Gijs)

Tracking

Trunk
Firefox 46
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox46 fixed)

Details

Attachments

(1 attachment)

Assignee

Description

4 years ago
No description provided.
Assignee

Comment 2

4 years ago
Comment on attachment 8699539 [details]
MozReview Request: Bug 1233470 - make browser/modules eslintable, r?Mossop

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/28271/diff/1-2/
Assignee

Comment 3

4 years ago
Comment on attachment 8699539 [details]
MozReview Request: Bug 1233470 - make browser/modules eslintable, r?Mossop

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/28271/diff/2-3/
Assignee

Updated

4 years ago
Blocks: 1233537
Assignee

Comment 4

4 years ago
Did we not turn on the windows line endings rule?  browser/modules/test/browser_BrowserUITelemetry_buckets.js  seems to be using them, but eslint didn't mention it. I converted it to unix in the latest revision of this patch anyway, but...
No longer blocks: 1233537
Assignee

Updated

4 years ago
Blocks: 1233537
Comment on attachment 8699539 [details]
MozReview Request: Bug 1233470 - make browser/modules eslintable, r?Mossop

https://reviewboard.mozilla.org/r/28271/#review25337

Awesome, thanks

::: browser/modules/Social.jsm:142
(Diff revision 3)
> +        p.enabled = enable;

providers already check whether they are enabled so you can just blindly set p.enabled to enabled and save a check.
Attachment #8699539 - Flags: review?(dtownsend) → review+

Comment 7

4 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/b1db238e620f
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 46

Updated

4 years ago
Depends on: 1234398

Comment 8

3 years ago
[bugday-20160323]

Status: RESOLVED,FIXED -> UNVERIFIED

Comments:
STR: Not clear.
Developer specific testing

Component: 
Name			Firefox
Version			46.0b9
Build ID		20160322075646
Update Channel	beta
User Agent		Mozilla/5.0 (Windows NT 6.1; WOW64; rv:46.0) Gecko/20100101 Firefox/46.0
OS				Windows 7 SP1 x86_64

Expected Results: 
Developer specific testing

Actual Results: 
As expected
You need to log in before you can comment on or make changes to this bug.