Closed
Bug 1737980
Opened 3 years ago
Closed 2 years ago
Remove lodash omit usage
Categories
(DevTools :: Debugger, task)
DevTools
Debugger
Tracking
(firefox98 fixed)
RESOLVED
FIXED
98 Branch
Tracking | Status | |
---|---|---|
firefox98 | --- | fixed |
People
(Reporter: nchevobbe, Assigned: nchevobbe)
References
Details
(Keywords: good-first-bug)
Attachments
(1 file)
Here we can probably replace this with the spread operator + delete
prefs.expressions = expressions.map(expression => {
const expressionWithoutValue = {...expression};
delete expressionWithoutValue.value;
return expression;
})
Assignee | ||
Comment 1•2 years ago
|
||
It was only used in storeExpressions, where we can replace it by storing only the input property
instead of removing the value property from the object we store.
Depends on D136096
Updated•2 years ago
|
Assignee: nobody → nchevobbe
Status: NEW → ASSIGNED
Pushed by nchevobbe@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/f0e1f377dd9e [devtools] Remove lodash omit usage. r=bomsy.
Comment 3•2 years ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
status-firefox98:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 98 Branch
You need to log in
before you can comment on or make changes to this bug.
Description
•