Closed
Bug 1301316
Opened 8 years ago
Closed 8 years ago
Run stylo reftest in TaskCluster
Categories
(Core :: CSS Parsing and Computation, defect)
Core
CSS Parsing and Computation
Tracking
()
RESOLVED
FIXED
People
(Reporter: shinglyu, Assigned: shinglyu)
References
Details
Attachments
(2 files)
(deleted),
patch
|
Details | Diff | Splinter Review | |
(deleted),
text/x-review-board-request
|
heycam
:
review+
|
Details |
After bug 1288350 merge into the stylo-incubator repo, we want to run it in TaskCluster for try pushes.
Assignee | ||
Comment 1•8 years ago
|
||
This is just my random hack based on studying the code, not sure how to test it.
Attachment #8789249 -
Flags: review?(cam)
Assignee | ||
Comment 2•8 years ago
|
||
Based on the discussion today, we might want to land this into stylo-incubator directly.
Comment 3•8 years ago
|
||
Yeah, I have no idea about the TC description format. Once you land the manifests to the stylo repo, you can do a push to try with this TC patch to see if it works.
Assignee | ||
Updated•8 years ago
|
Attachment #8789249 -
Flags: review?(cam)
Assignee | ||
Comment 4•8 years ago
|
||
I'm trying to do a try run and see if it runs and get a `reftest-stylo.list not found` error
https://treeherder.mozilla.org/#/jobs?repo=try&revision=2ff397b36b0f
Comment hidden (mozreview-request) |
Assignee | ||
Updated•8 years ago
|
Attachment #8789266 -
Flags: review?(cam)
Comment 6•8 years ago
|
||
mozreview-review |
Comment on attachment 8789266 [details]
Bug 1301316 - Run stylo reftest in TaskCluster
https://reviewboard.mozilla.org/r/77562/#review76110
Since it seems to work, r=me to land this on the stylo branch.
Attachment #8789266 -
Flags: review?(cam) → review+
Comment 7•8 years ago
|
||
Manish, did you fix this bug? Are the incubator/stylo reftests running in TaskCluster now?
Flags: needinfo?(manishearth)
Comment 8•8 years ago
|
||
Yes, Shing's patches are in the incubator now. This can't be landed on m-c until the rest of the CI stuff gets there.
Assignee: nobody → slyu
Status: NEW → RESOLVED
Closed: 8 years ago
Flags: needinfo?(manishearth)
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•