Response schemas: set all keys as required, add `additionalProperties = false` flag

[gr2m]

Response schemas: set all keys as required, add `additionalProperties = false` flag

[gr2m]

Make sure to add the keys in response schema overrides in workarounds.js

[RomanHotsiy]

@gr2m what about some keys that were generated from null?

e.g. here https://developer.github.com/v3/repos/#response template_repository is null in examples so I don't think it should be marked as required.

Btw, template_repository in this particular case is turned into type: string property which I suppose is another issue.

[gr2m]

right now, none of the keys are marked as required, which is not ideal either.

e.g. here https://developer.github.com/v3/repos/#response template_repository is null in examples so I don't think it should be marked as required.

I don't see the template_repository in the response?

[RomanHotsiy]

[gr2m]

I see it now. Sorry, odd that I missed it before.

Anyway, there is no ideal solution to this right now. We can add more workarounds to /lib/endpoint/overrides/workarounds.js on a case-to-case basis. But I think setting everything to required is still the better default.

Btw, template_repository in this particular case is turned into type: string property which I suppose is another issue

yeah, that seems unrelated