web-platform-tests / wpt.fyi

web-platform-tests dashboard
https://wpt.fyi/
Other
183 stars 88 forks source link

Include webkitgtk in /api/runs #1594

Open foolip opened 4 years ago

foolip commented 4 years ago

https://wpt.fyi/api/runs?product=webkitgtk returns runs, but they're not included by default.

Alternatively, for someone who wants to clone all the data (https://github.com/web-platform-tests/wpt.fyi/issues/636) perhaps another API is needed to list what products there exists runs for?

stephenmcgruer commented 4 years ago

So it was added deliberately as not a default browser in 36f282424637919319be034109ca6513a974e340

What do we think is the bar for browsers showing up in the various APIs? The ones that I know about that load defaults are api/runs and api/interop.

cc @Hexcles for the above.

I am also a bit concerned that api/search may also use api/runs if no product is selected, and if the UI calls api/search without specifying runs it may start including webkitgtk data even when the user hasn't selected it (since it isn't a default product there). Need to test that.

foolip commented 4 years ago

Given an API to list existing products we could probably have no default set of runs or at least treat it as legacy and not depend on it in the wpt.fyi front end.