Closed
Bug 1156145
Opened 10 years ago
Closed 10 years ago
[Settings] cleaned up settings related scripts in xfail.list
Categories
(Firefox OS Graveyard :: Gaia::Settings, defect)
Tracking
(b2g-master fixed)
RESOLVED
FIXED
Tracking | Status | |
---|---|---|
b2g-master | --- | fixed |
People
(Reporter: eragonj, Assigned: eragonj)
References
Details
Attachments
(1 file)
As title, in this phase, it would be nice if we can clean up some old codes in xfail.list.
Comment 1•10 years ago
|
||
Assignee | ||
Comment 2•10 years ago
|
||
Comment on attachment 8594648 [details] [review]
[gaia] EragonJ:bug-1156145 > mozilla-b2g:master
Hi all, this is a simple refactor works trying to remove all settings related files from xfail.list. Please help me review related parts that you guys participate in before ! Thanks !
@Ian, need your help for bluetooth.js
@Arthur, need your help for the other parts in Settings app.
Attachment #8594648 -
Flags: review?(iliu)
Attachment #8594648 -
Flags: review?(arthur.chen)
Comment 3•10 years ago
|
||
Comment on attachment 8594648 [details] [review]
[gaia] EragonJ:bug-1156145 > mozilla-b2g:master
In part of bluetooth.js, r+ with me. And I leave one comment which is relative with indent on GitHub. Please take a look for readable style. Thanks for EJ's effort here:)
Attachment #8594648 -
Flags: review?(iliu) → review+
Comment 4•10 years ago
|
||
Comment on attachment 8594648 [details] [review]
[gaia] EragonJ:bug-1156145 > mozilla-b2g:master
Thanks for the clean up. r=me with the comments addressed.
Attachment #8594648 -
Flags: review?(arthur.chen) → review+
Assignee | ||
Updated•10 years ago
|
Keywords: checkin-needed
Updated•10 years ago
|
Keywords: checkin-needed
Comment 6•10 years ago
|
||
http://docs.taskcluster.net/tools/task-graph-inspector/#IQjOWOi-TSeLAmyPAuXNSQ
The pull request failed to pass integration tests. It could not be landed, please try again.
Assignee | ||
Updated•10 years ago
|
Keywords: checkin-needed
Updated•10 years ago
|
Keywords: checkin-needed
Comment 7•10 years ago
|
||
Pull request has landed in master: https://github.com/mozilla-b2g/gaia/commit/dc24751bb9f00046805f4294683b3e6712baed26
Updated•10 years ago
|
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Assignee | ||
Updated•10 years ago
|
status-b2g-master:
--- → fixed
You need to log in
before you can comment on or make changes to this bug.
Description
•