Closed
Bug 791625
Opened 13 years ago
Closed 13 years ago
Remove Mozmill IDE from the extension
Categories
(Testing Graveyard :: Mozmill, defect)
Testing Graveyard
Mozmill
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: whimboo, Assigned: whimboo)
Details
(Whiteboard: [mozmill-2.0+])
Attachments
(1 file)
Given that we want to get rid of the extension I'm voting to remove it entirely from the code base for Mozmill 2.0.
|
Assignee | |
Comment 1•13 years ago
|
||
As agreed in the meeting lets get it removed.
Whiteboard: [mozmill-2.0?] → [mozmill-2.0+]
|
|
Assignee | |
Updated•13 years ago
|
Whiteboard: [mozmill-2.0+] → [mozmill-2.0+][mentor=whimboo][lang=js]
|
|
Assignee | |
Comment 2•13 years ago
|
||
I will take it so all the code can get removed soon.
Assignee: nobody → hskupin
Status: NEW → ASSIGNED
Whiteboard: [mozmill-2.0+][mentor=whimboo][lang=js] → [mozmill-2.0+]
|
|
Assignee | |
Comment 3•13 years ago
|
||
Clint, do you want to leave the inspector in the extension or take it out too? I would prefer the latter because we do no longer have an entry point for it.
|
|
Assignee | |
Comment 4•13 years ago
|
||
Pointer to Github pull-request
|
|
Assignee | |
Comment 5•13 years ago
|
||
Comment on attachment 663082 [details]
Patch v1
Kill it! That removes all traces from the IDE. No files, methods, or variables should be left.
Attachment #663082 -
Attachment description: Pointer to Github pull request: https://github.com/mozilla/mozmill/pull/101 → Patch v1
Attachment #663082 -
Flags: review?(ctalbert)
Comment on attachment 663082 [details]
Patch v1
If we're removing the UI let's remove all the UI, including the inspector. If we need to fish the code out to generate something, we have source control.
Attachment #663082 -
Flags: review?(ctalbert) → review+
|
|
Assignee | |
Comment 7•13 years ago
|
||
OMG it's done! No more IDE is around...
https://github.com/mozilla/mozmill/commit/e60624bfe79bcd27f24dcc7ba80adcf8c593052c
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
|
||
Comment 8•13 years ago
|
||
(In reply to Henrik Skupin (:whimboo) from comment #7)
> OMG it's done! No more IDE is around...
>
> https://github.com/mozilla/mozmill/commit/
> e60624bfe79bcd27f24dcc7ba80adcf8c593052c
There are lot of IDE related bug.
What will You do with those?
If I were You, I would resolve those as INVALID
and let those block this bug. :)
|
|
Assignee | |
Comment 9•13 years ago
|
||
Oh, right. Nearly missed that step. Thanks for pointing that out. I will walk through each of those in a bit.
|
||
Updated•9 years ago
|
Product: Testing → Testing Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•