Closed
Bug 1554875
Opened 6 years ago
Closed 6 years ago
Simplify devtools/client/inspector/markup/markup.js and remove eslint-disable complexity
Categories
(DevTools :: Inspector, task, P3)
DevTools
Inspector
Tracking
(firefox70 fixed)
RESOLVED
FIXED
Firefox 70
| Tracking | Status | |
|---|---|---|
| firefox70 | --- | fixed |
People
(Reporter: pbro, Assigned: arunmohandm)
References
(Blocks 1 open bug)
Details
Attachments
(1 file)
This file contains a special eslint comment to temporarily disable the complexity eslint rule since it contains one or more functions that exceed the maximum complexity threshold. The code should be simplified, and the comment removed. Link to the code in question: https://searchfox.org/mozilla-central/search?q=eslint-disable+complexity&case=false®exp=false&path=devtools%2Fclient%2Finspector%2Fmarkup%2Fmarkup.js
|
Reporter | |
Updated•6 years ago
|
Component: General → Inspector
|
||
Updated•6 years ago
|
Priority: -- → P3
|
Assignee | |
Comment 1•6 years ago
|
||
Pushed by aciure@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/6e3e96412fd9
Refactor _onShortcut method to remove disabled complexity eslint rule. r=pbro
|
||
Comment 3•6 years ago
|
||
| bugherder | ||
Status: NEW → RESOLVED
Closed: 6 years ago
status-firefox70:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 70
|
||
Updated•6 years ago
|
Assignee: nobody → arunmohandm
You need to log in
before you can comment on or make changes to this bug.
Description
•