Closed Bug 1330099 Opened 8 years ago Closed 2 years ago

Enable object-shorthand eslint rule

Categories

(DevTools :: General, enhancement, P3)

enhancement

Tracking

(firefox105 fixed)

RESOLVED FIXED
105 Branch
Tracking Status
firefox105 --- fixed

People

(Reporter: ntim, Assigned: standard8)

References

(Blocks 2 open bugs)

Details

Attachments

(3 files)

      No description provided.
Severity: normal → enhancement
Has Regression Range: --- → irrelevant
Has STR: --- → irrelevant
Priority: -- → P3
Comment on attachment 8825567 [details]
Bug 1330099 - Enable object-shorthand eslint rule.

https://reviewboard.mozilla.org/r/103694/#review104652

Great, this clean up looks good to me.  Thanks for working on it!
Attachment #8825567 - Flags: review?(jryans) → review+
Pushed by ntim.bugs@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/6bc38f188ef2
Enable object-shorthand eslint rule. r=jryans
Backout by archaeopteryx@coole-files.de:
https://hg.mozilla.org/integration/autoland/rev/472ee679901a
Backed out changeset 6bc38f188ef2 for devtools failures. r=backout
Assignee: ntim.bugs → nobody
Product: Firefox → DevTools
Blocks: 1430596
Assignee: nobody → standard8
Status: NEW → ASSIGNED
Pushed by mbanner@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/4bff5e3a18ce
Disable ESlint rule object-shorthand in devtools tests where required to keep tests passing. r=jdescottes
https://hg.mozilla.org/integration/autoland/rev/9c392ddae696
Enable object-shorthand eslint rule. r=jdescottes
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → 105 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: