Closed Bug 1821621 Opened 2 years ago Closed 2 years ago

Convert toolkit/components/kvstore to ES modules

Categories

(Toolkit :: Storage, task)

task

Tracking

()

RESOLVED FIXED
113 Branch
Tracking Status
firefox113 --- fixed

People

(Reporter: standard8, Assigned: tiwari.abhishektiwari23, Mentored)

References

(Blocks 1 open bug)

Details

(Keywords: good-first-bug, Whiteboard: [esmification-timeline][lang=js])

Attachments

(1 file, 1 obsolete file)

I'm happy to mentor work towards this bug. Please don't ask to be assigned to it, though you can say you're working on it. You'll be automatically assigned when you upload the first patches.

Here's what to do:

  • Make sure you have a local build of Firefox up and running. Use these instructions if you haven't already.
  • Using the ESMification walkthrough as a guide:
    1. Run ./mach esmify --convert path/to/files
    2. Examine the output for any errors from ESLint and fix them.
    3. Next convert the imports - ./mach esmify --import . --prefix path/to/files
    4. Again, check for any errors and fix them.
    5. Run a build to check it passes.
    6. Also run ./mach eslint --fix path/to/files and check there are no raised failures
    7. Commit the changes with a commit messages such as "Bug nnnnnn - Convert path/to/files to ES modules. r?Standard8"
  • Use moz-phab submit to submit the patches for review.
Assignee: nobody → tiwari.abhishektiwari23
Status: NEW → ASSIGNED

Depends on D172233

Attachment #9322336 - Attachment description: Bug 1821621 - Convert toolkit/components/kvstore to ES modules r=standard8 → WIP: Bug 1821621 - Convert toolkit/components/kvstore to ES modules updated rstandard8
Attachment #9322336 - Attachment description: WIP: Bug 1821621 - Convert toolkit/components/kvstore to ES modules updated rstandard8 → Bug 1821621 - Convert toolkit/components/kvstore to ES modules r=standard8
Attachment #9322337 - Attachment is obsolete: true
Pushed by mbanner@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/27e61cf5eac9 Convert toolkit/components/kvstore to ES modules r=Standard8,extension-reviewers,robwu
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → 113 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: