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)

x86
macOS
defect
Not set
normal

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 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 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 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+
Keywords: checkin-needed
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.
Keywords: checkin-needed
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: