-
Notifications
You must be signed in to change notification settings - Fork 92
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
api/results documentation does not reflect current behaviour #2940
Comments
I'd also like to know what "s", "O", and "c" represent. This was made more confusing by gh-2920 because the documentation now claims that the old format is "v2":
Have any maintainers recognized this issue? |
The new format was introduced in gh-2858. I'm not an authority on this, but here's my interpretation in case it helps other folks in the interim. "s" is for "status", and "c" is for "counts":
|
Hey everyone. Apologies for the delay in response. @jugglinmike's description above of the changes here is exactly correct. The intent of the change was to separate the overall test status from the fraction of passes and failures to help display more accurate information on wpt.fyi. Thank you for pointing out the discrepancy in the documentation. 🙂 I'm submitting a change to the docs to reflect this change. |
In https://github.com/web-platform-tests/wpt.fyi/tree/main/api#apiresults, it says
while it is actually something like that
"/path/to/test.html": {"s": "O", "c": [1, 1]},
It would be good to update documentation
The text was updated successfully, but these errors were encountered: