Closed
Bug 1953970
Opened 1 year ago
Closed 1 year ago
Prepare parts of toolkit/components/places for TypeScript
Categories
(Toolkit :: Places, task, P3)
Toolkit
Places
Tracking
()
RESOLVED
FIXED
138 Branch
| Tracking | Status | |
|---|---|---|
| firefox138 | --- | fixed |
People
(Reporter: standard8, Assigned: standard8)
References
Details
Attachments
(4 external files)
This is a collection of changes that we'll be using as we set-up TypeScript on the toolkit/components/places directory.
| Assignee | ||
Comment 1•1 year ago
|
||
This applies the ESLint fixes to the current production code.
| Assignee | ||
Comment 2•1 year ago
|
||
This defines the appropraite types so that TypeScript knows about the metrics definitions.
| Assignee | ||
Comment 3•1 year ago
|
||
| Assignee | ||
Comment 4•1 year ago
|
||
This also changes Sqlite.sys.mjs so that TypeScript can use the OpenedConnection type from SyncedBookmarksMirror.sys.mjs.
Pushed by mbanner@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/4ff849c057b2
Automatically fix issues relating to ESLint mozilla/valid-jsdoc configuration rules in toolkit places code. r=places-reviewers,daisuke
https://hg.mozilla.org/integration/autoland/rev/18119f894dfc
Add Glean typescript definitions for toolkit places metrics. r=places-reviewers,daisuke
https://hg.mozilla.org/integration/autoland/rev/16e7d5e27d59
Use cenums in mozISyncedBookmarksMirror to make it easier for TypeScript to handle the constants. r=places-reviewers,daisuke
https://hg.mozilla.org/integration/autoland/rev/27b398e13a67
Fix TypeScript issues raised in BookmarkHTMLUtils.sys.mjs and SyncedBookmarksMirror.sys.mjs. r=places-reviewers,daisuke.
Comment 6•1 year ago
|
||
| bugherder | ||
https://hg.mozilla.org/mozilla-central/rev/4ff849c057b2
https://hg.mozilla.org/mozilla-central/rev/18119f894dfc
https://hg.mozilla.org/mozilla-central/rev/16e7d5e27d59
https://hg.mozilla.org/mozilla-central/rev/27b398e13a67
Status: NEW → RESOLVED
Closed: 1 year ago
status-firefox138:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 138 Branch
You need to log in
before you can comment on or make changes to this bug.
Description
•