fix: add override for bad usage of anyOf

[wolfy1339]

See octokit/types.ts#534

---

Behavior

Before the change?

• The OpenAPI spec had bad usage of anyOf, trying to use it to make one property required at a time (either or)

After the change?

• Use a oneOf instead, as both properties can't be present at the same time, and put the different properties in 2 schemas inside of that oneOf
• The OpenAPI spec can now result in proper types

Other information

• This was probably a quirk that was removed in openapi-typescript v6, as they went more strict on the openapi spec parsing
• Filed a bug upstream [Schema Inaccuracy] Bad usage of anyOf to require either or of certain properties in definitions github/rest-api-description#2510

Additional info

Pull request checklist

• Tests for the changes have been added (for bug fixes / features)
• Docs have been reviewed and added / updated if needed (for bug fixes / features)
• Added the appropriate label for the given change

Does this introduce a breaking change?

Please see our docs on breaking changes to help!

• Yes (Please add the Type: Breaking change label)
• No

If Yes, what's the impact:

• N/A

Pull request type

Please add the corresponding label for change this PR introduces:

• Bugfix: Type: Bug

---

[wolfy1339]

For easier reviewing, since GitHub doesn't want to show the diff online, or generate a diff/patch file, here are the diffs for each commit.

0001-fix-add-override-for-bad-usage-of-anyOf.txt
0002-build-generate-openapi-specs.txt
0003-fix-add-override-for-bad-anyOf-in-pulls-request-revi.txt
0004-build-generate-openapi-specs.txt

[kfcampbell]

It's unclear to me why the diff isn't displaying. The changes in your linked text files look reasonable to me as a workaround while we wait for the upstream issue to be fixed.

[octokitbot]

🎉 This PR is included in version 11.1.1 🎉

The release is available on:

• GitHub release
• npm package (@latest dist-tag)

Your semantic-release bot 📦🚀