-
Notifications
You must be signed in to change notification settings - Fork 251
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
[REST] avatar_url examples are misleading #3259
Comments
Hey team, happy to pick it up. 😄 |
fwiw, REST content can't be fixed by externals |
@jsoref Did you ran your spell-checker for GitHub Docs as well? like others? 😁 |
Yes. And most of my fixes are merged. |
@jsoref Thank you for opening this issue! I'll get this triaged for review ✨ |
Thank you for opening this issue! Changes to the REST API schema can be requested in github/rest-api-description. I will transfer your issue over to that open source repo. |
Code of Conduct
What article on docs.github.com is affected?
https://docs.github.com/en/rest/commits/statuses?apiVersion=2022-11-28
What part(s) of the article would you like to see updated?
The example responses have things like:
Note that both
avatar_url
s are 404:This doesn't seem to match the real world, and since I'm looking for the word
error
, it's really confusing/misleading.Checking https://api.github.com/users/octocat, it appears it's more likely that it should be something like:
I can't tell if that's the
avatar_url
I should see in both places or if they're very different things.Additional information
I'm trying to get information on
error
,failure
,pending
, orsuccess
, so I'm looking for those in the output of responses...Note that the use of a lot of
"id": 1,
for things that might be different is very counter-productive.It'd be really nice if the sample output covered each of the status states instead of just one.
The text was updated successfully, but these errors were encountered: