Closed Bug 1681890 Opened 6 months ago Closed 5 months ago

Remove defer usage in devtools/client/shared/components/test/chrome/head.js

Categories

(DevTools :: Shared Components, task, P3)

task

Tracking

(firefox86 fixed)

RESOLVED FIXED
86 Branch
Tracking Status
firefox86 --- fixed

People

(Reporter: Honza, Assigned: mick, Mentored)

References

(Blocks 1 open bug)

Details

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

Attachments

(1 file)

We should use new Promise instead.

Honza

hi
I want to work on this bug.

Abhishek

Hi Abishek,

If you're interested in working in this bug, feel free to submit a patch and the bug will be assigned automatically.

Simply remove this line: https://searchfox.org/mozilla-central/rev/0eebf69d04cc0cdd252b135dda2fa9b7c2fa2bfb/devtools/client/shared/components/test/chrome/head.js#17

See https://firefox-source-docs.mozilla.org/setup/contributing_code.html for steps to submit a patch.

Let me know if you've got any questions.

Mentor: ntim.bugs
Whiteboard: [lang=js]

hi, can i work on this?

sir,how to download the source code of ' https://searchfox.org/mozilla-central/rev/0eebf69d04cc0cdd252b135dda2fa9b7c2fa2bfb/devtools/client/shared/components/test/chrome/head.js#17' since there is no github link provided. Bcoz once we found github page of this,we can clone to our local machine and then change the code and make a pull request .Please help

hi
I can do it . Thanks for guidance.

Abhishek

Hi Tim,
I remove the line but problem arises when I submit the patch. I use git commands and i also installed a tool called moz-phab. But when i run it on git bash its shows command not found. I installed moz-phab in c:\mozilla-build\start-shell.bat. What should i do ?
Can u guide me?

Flags: needinfo?(ntim.bugs)

(In reply to Abhishek Sharma from comment #6)

Hi Tim,
I remove the line but problem arises when I submit the patch. I use git commands and i also installed a tool called moz-phab. But when i run it on git bash its shows command not found. I installed moz-phab in c:\mozilla-build\start-shell.bat. What should i do ?
Can u guide me?

I personally use Mercurial (hg), the standard Firefox workflow. I'm not too familiar with things on Windows generally.

Usually my workflow is:

  1. Run hg up central --clean if not already done
  2. Make the changes
  3. Run hg commit -m "Bug 1681890 - Remove defer import in devtools/client/shared/components/test/chrome/head.js"
  4. Run moz-phab

You might want to chat with folks on the #introduction channel on https://chat.mozilla.org for more specific help :)

Flags: needinfo?(ntim.bugs)

ok thanks for help.

Hi Tim ,
I solve the bug and commit it and all commands are working properly in Mercurial( hg ) but in last when I run moz-phab it shows missing .arrconfig file. Can u guide me ?
What should i do ?

where i can add .arrconfig file ?

I'm not sure, I think the "Conduit" channel on https://chat.mozilla.org is more familiar with this (but I'm sure "Introduction" can help as well).

Thanks Tim . I asked in Conduit Channel

Hi Tim,
Now it works .I have one question, who is the reviewer of this bug. So I add in my commit.

I submit my patch and remove the line from the defer . Can you see it?

(In reply to Abhishek Sharma from comment #14)

I submit my patch and remove the line from the defer . Can you see it?

I can't, your Phabricator activity looks empty: https://phabricator.services.mozilla.com/p/mick.sharma02/

and there's no listing on Bugzilla either.

(In reply to Abhishek Sharma from comment #13)

Hi Tim,
Now it works .I have one question, who is the reviewer of this bug. So I add in my commit.

I'm happy to review this one, since it's pretty straightforward.

Hi Tim,
Sorry for late reply. For two days , my health condition is not so good. I check it once again than commit it. Hope you understand.

hi Tim,may i work on this?
i have done the stuff ,only commiting the patch left. @Abishek do u have any objection regarding this? since you the one first asked Tim to work on this bug.If you have any discomfort regarding this..please feel free to tell me I left over to you completely.If not,so that i can submit the patch naa.

Thank you
with a humble,
Dhanesh

If any of you submit a patch, the bug will automatically be assigned to you. There are always other good-first-bugs if the other person doesn't get to it.

Hi Tim,
I have a question , when I write your name as a reviewer it says Tim not a valid reviewer name. What name should I write ? or may I write ntim.
Can you please guide ?

Assignee: nobody → mick.sharma02
Status: NEW → ASSIGNED

Hi Tim,
I submit my patch . Can you please review it.

Attachment #9195596 - Attachment description: "Bug 1681890 - [devtools] Remove defer import in devtools/client/shared/components/test/chrome/head.js r=ntim" → Bug 1681890 - [devtools] Remove defer import in devtools/client/shared/components/test/chrome/head.js r=ntim
Pushed by ntim.bugs@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/ea7929f39db5
[devtools] Remove defer import in devtools/client/shared/components/test/chrome/head.js r=ntim

Hi Tim,
I want to solve more bugs , where should I find?
Can you help me?

Status: ASSIGNED → RESOLVED
Closed: 5 months ago
Resolution: --- → FIXED
Target Milestone: --- → 86 Branch

(In reply to Abhishek Sharma from comment #24)

Hi Tim,
I want to solve more bugs , where should I find?
Can you help me?

Bug 1685638 might be interesting.

Attachment #9195596 - Attachment description: Bug 1681890 - [devtools] Remove defer import in devtools/client/shared/components/test/chrome/head.js r=ntim → "Bug 1681890 - [devtools] Remove defer import in devtools/client/shared/components/test/chrome/head.js r=ntim"
You need to log in before you can comment on or make changes to this bug.