Closed Bug 1820878 Opened 2 years ago Closed 2 years ago

Convert toolkit/mozapps/downloads to ES modules

Categories

(Toolkit :: Downloads API, task, P3)

task

Tracking

()

RESOLVED FIXED
114 Branch
Tracking Status
firefox114 --- fixed

People

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

References

(Blocks 1 open bug)

Details

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

Attachments

(2 files, 2 obsolete files)

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. Commit the changes with a commit messages such as "Bug nnnnnn - Convert path/to/files to ES modules. r?Standard8"
    4. Next convert the imports - ./mach esmify --import . --prefix path/to/files
    5. Again, check for any errors and fix them.
    6. Run a build to check it passes.
    7. Also run ./mach eslint --fix path/to/files and check there are no raised failures
    8. Commit the changes with a commit message such as "Bug nnnnnn - Convert consumers of path/to/files to import ES modules directly. r?Standard8"
  • Use moz-phab submit to submit the patches for review.
Whiteboard: [esmification-timeline] → [esmification-timeline][lang=js]
Mentor: standard8

Depends on D172024

Assignee: nobody → tiwari.abhishektiwari23
Status: NEW → ASSIGNED

Depends on D172049

I have made all the changes, but I still think it requires some changes. I will work on them, and the number of commits is high because the error becomes visible only after making the commits, so please don't mind.
Thanks

I have deleted the file I was working on, so I am making a new patch for it. Thanks

Attachment #9321970 - Attachment is obsolete: true
Attachment #9321969 - Attachment is obsolete: true
Severity: -- → S3
Priority: -- → P3

Hi Abhishek, are you still working on this? If you need help with the rebase, please ask in #introduction on Matrix.

Flags: needinfo?(tiwari.abhishektiwari23)

I apologize for the delay in getting back to you. Last week, I was mostly inactive due to my exam in Infrastructure Economics, which took place today. (https://onlinecourses.nptel.ac.in/noc23_mg25/announcements?force=true) Now that the exam is over, I am available and will be able to handle this issue promptly. Thank you for your understanding.

Flags: needinfo?(tiwari.abhishektiwari23)
Attachment #9322366 - Attachment description: Bug 1820878 - Convert toolkit/mozapps/downloads to ESmodules . r=Standard8 → Bug 1820878 - Convert toolkit/mozapps/downloads to ESmodules . r=Standard8!
Pushed by mbanner@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/e9a3651b0969 Convert toolkit/mozapps/downloads to ESmodules . r=Standard8,extension-reviewers,application-update-reviewers,bytesized,robwu
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → 114 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: