Closed Bug 1156145 Opened 9 years ago Closed 9 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+
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.
Status: NEW → RESOLVED
Closed: 9 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: