Convert toolkit/mozapps/update to ES modules
Categories
(Toolkit :: Application Update, task, P3)
Tracking
()
Tracking | Status | |
---|---|---|
firefox113 | --- | fixed |
People
(Reporter: standard8, Assigned: yasminekaltouma, Mentored)
References
(Blocks 1 open bug)
Details
(Keywords: good-first-bug, Whiteboard: [esmification-timeline][lang=js])
Attachments
(2 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.
- If you need help setting up, ask in #introduction on Matrix
- Using the ESMification walkthrough as a guide:
- Run
./mach esmify --convert path/to/files
- Examine the output for any errors from ESLint and fix them.
- Commit the changes with a commit messages such as "Bug nnnnnn - Convert path/to/files to ES modules. r?Standard8"
- Next convert the imports -
./mach esmify --import . --prefix path/to/files
- Again, check for any errors and fix them.
- Run a build to check it passes.
- Also run
./mach eslint --fix path/to/files
and check there are no raised failures - Commit the changes with a commit message such as "Bug nnnnnn - Convert consumers of path/to/files to import ES modules directly. r?Standard8"
- Run
- Use
moz-phab submit
to submit the patches for review.
Assignee | ||
Comment 1•1 year ago
|
||
Hi, I am Leila kaltouma, an Outreachy applicant. I'm currently working on this bug. Thank you for offering to mentor.
Hey. I'm Siya, an Outreachy applicant. I would like to work on this issue and am currently working on it.
Reporter | ||
Comment 3•1 year ago
|
||
(In reply to Siya from comment #2)
Hey. I'm Siya, an Outreachy applicant. I would like to work on this issue and am currently working on it.
Hi Siya, please Leila a chance to put up the patch as they commented first. There should be other bugs around and available (I'll probably also be filing more bugs later).
Comment 4•1 year ago
|
||
Hello Mark. My name is Yusuf, and i am also an outreachy applicant participating for outreachy internship. I would like to be assigned to work for the upcoming bugs.
Comment 5•1 year ago
|
||
And can i also be assigned to work on this issue?
Reporter | ||
Comment 6•1 year ago
|
||
(In reply to yusufjakada5 from comment #5)
And can i also be assigned to work on this issue?
Please make sure you read the previous comments when commenting on bugs. As you'll see Leila is already working on this, and I have already suggested to Siya that they find a different bug.
Reporter | ||
Updated•1 year ago
|
Assignee | ||
Comment 7•1 year ago
|
||
Assignee | ||
Comment 8•1 year ago
|
||
Depends on D172120
Assignee | ||
Comment 9•1 year ago
|
||
Thank you for assigning this task to me.
Updated•1 year ago
|
Comment 10•1 year ago
|
||
Pushed by mbanner@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/8da6d504d011 Convert toolkit/mozapps/update to ES modules. r=Standard8,application-update-reviewers,bytesized,nalexander https://hg.mozilla.org/integration/autoland/rev/190fed1de1bf Convert toolkit/mozapps/update to import ES modules directly. r=Standard8,settings-reviewers,application-update-reviewers,bytesized
Comment 11•1 year ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/8da6d504d011
https://hg.mozilla.org/mozilla-central/rev/190fed1de1bf
Description
•