Closed
Bug 1041535
Opened 10 years ago
Closed 10 years ago
Add web-platform-tests to trychooser
Categories
(Release Engineering :: General, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: jgraham, Assigned: jgriffin)
References
Details
(Keywords: trychooser)
Attachments
(1 file, 2 obsolete files)
|
4.24 KB,
patch
|
emorley
:
review+
|
Details | Diff | Splinter Review |
Once web-platform-tests are enabled on try, they should be added as an option in trychooser.
|
||
Updated•10 years ago
|
Keywords: trychooser
|
Reporter | |
Comment 1•10 years ago
|
||
Can't land until web-platform-tests is enabled on Try, obviously. Fixesa bug where it was assumed that there was only one group of unittests, so checking any one would enable all groups.
Attachment #8475968 -
Flags: review?(emorley)
|
|
Reporter | |
Updated•10 years ago
|
Assignee: nobody → james
Status: NEW → ASSIGNED
|
|
||
Comment 2•10 years ago
|
||
Comment on attachment 8475968 [details] [diff] [review]
Add web-platform-tests to trychooser
Thank you :-)
Attachment #8475968 -
Flags: review?(emorley) → review+
|
Assignee | |
Comment 3•10 years ago
|
||
Oops, you're right, the first patch has the effect of disabling wpt-chunked on cedar except for linux64, which is enabled globally. This adds it back.
Attachment #8478574 -
Flags: review?(nthomas)
|
|
Assignee | |
Updated•10 years ago
|
Assignee: james → jgriffin
|
|
Assignee | |
Updated•10 years ago
|
Attachment #8478574 -
Attachment is obsolete: true
Attachment #8478574 -
Flags: review?(nthomas)
|
||
Comment 5•10 years ago
|
||
Comment on attachment 8478575 [details] [diff] [review]
Re-enable wpt-chunked on cedar for non-linux64 platforms,
Please put a comment on bug 1056973 to say the followup ended up here.
Attachment #8478575 -
Flags: review?(nthomas) → review+
|
|
Assignee | |
Comment 6•10 years ago
|
||
Comment on attachment 8478575 [details] [diff] [review]
Re-enable wpt-chunked on cedar for non-linux64 platforms,
Whoops, attached this to wrong bug, I'll put in on the right one. :)
Attachment #8478575 -
Attachment is obsolete: true
|
|
Reporter | |
Comment 7•10 years ago
|
||
|
|
Reporter | |
Comment 8•10 years ago
|
||
I'm not sure what's needed to update the website?
|
|
||
Comment 9•10 years ago
|
||
(In reply to James Graham [:jgraham] from comment #8)
> I'm not sure what's needed to update the website?
Another bug to deploy it, eg bug 1059236
|
|
Assignee | |
Updated•10 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
|
||
Updated•8 years ago
|
Component: Tools → General
You need to log in
before you can comment on or make changes to this bug.
Description
•