[Schema Inaccuracy] Pull Request Files Changed `blob_url`, `raw_url`, and `sha` should be nullable

[bigfootjon]

Schema Inaccuracy

When a PR changes a submodule, the blob_url and raw_url are null, in some other cases that I can't pin down the sha field is null. However, the current schema marks these fields as non-nullable.

Expected

blob_url, raw_url, and sha in PR files changed responses should be marked as nullable.

Reproduction Steps

$ curl https://api.github.com/repos/pytorch/pytorch/pulls/89846/files

The above PR changes a submodule, the fields can be seen to be nullable there.

[0xpablo]

We are experiencing the same issue. This is a big problem on strongly typed languages like Swift where decoding will fail at runtime. We patched the spec manually but it would be great to get this fixed.

[nickfloyd]

Closing due to the issue becoming stale. Please re open if you feel like this still needs to be addressed.

[bigfootjon]

This is not completed, I can't reopen this issue so I will file a new one

[nickfloyd]

Thank you for reopening @bthomas2622 based on need, I'll follow up with the owning team to see where this is, apologies @bigfootjon for the inconvenance.

[nickfloyd]

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