[Schema Inaccuracy] Pull Request Review Comment `position` should be nullable

[nickfloyd]

Schema Inaccuracy

For each Pull Request Comment in the API, it has a position field attached. Some comments are not attached to a position so this field is null. The Schema does not reflect this nuance.

Expected

position in Pull Request review comments should be marked as nullable

Reproduction Steps

We have observed this in 2 different APIs, either listing PR review comments or fetching a specific comment:

$ curl https://api.github.com/repos/pytorch/tutorials/pulls/2120/comments
$ curl https://api.github.com/repos/pytorch/tutorials/pulls/comments/1024558033

In both cases, the position field is null:

...
"position": null,
...

[github-actions]

[This is an automated message] This issue may have been opened accidentally. I'm going to close it now, but feel free to open a new issue with a more descriptive title and body.

[nickfloyd]

Recreated on behalf of @bigfootjon - due to transfer tag issues.

[nickfloyd]

Hey @bigfootjon Thanks again for submitting an issue! I've routed this internal to the team that owns the endpoint so it can be addressed.