Simple User Type doesn't have to be null

[mlbonniec]

Checklist

• I checked all open bugs and none of them matched my problem.

Environment

• https://api.github.com
• GitHub Enterprise Server:
• GitHub Enterprise Cloud
• Node.js v14.15.1
• Browsers:
• Others:

Versions

├── @octokit/rest@18.5.2
├── @octokit/types@6.13.0

What happened?

On the schema, located at components['schemas']['full-repository'] (from '@octokit/openapi-types'), the value of owner can be null. However, a repo has to be owned, so null shouldn't be here.

Minimal test case to reproduce the problem

What did you expect to happen?

owner should be only an object, not an object or null

What the problem might be
Just remove null from the declaration file.

[wolfy1339]

This unfortunately isn't a bug that we can fix ourselves since those types are auto-generated.

I confirmed that this stems from Github's OpenAPI descriptions

I would file a bug report with them and link it back to here: https://github.com/github/rest-api-description/issues/new/choose
Then a staffer/contributor can confirm wether this is intended behaviour or an actual bug

[mlbonniec]

Ah, alright, thanks for your extremely quick answer!
I'll manage for the moment in my code, I hope it will be fixed quickly (if it's a bug)

[wolfy1339]

I filed an issue upstream with Github: github/rest-api-description#270

[AndrewSouthpaw]

Looks like the upstream issues may have been resolved, would it be possible to resume work on this?

[wolfy1339]

Can you try the latest version of @octokit/rest?

The v10.1.2 release of @octokit/types contains the fix.

[wolfy1339]

Closing as this has been fixed now.