Missing response properties on the branch protection api

[chris-smith-zocdoc]

There are several newer properties on the branch protection api that are missing

• required_signatures
• required_linear_history
• allow_force_pushes
• allow_deletions
• required_conversation_resolution

Relevant response model https://github.com/octokit/octokit.net/blob/main/Octokit/Models/Response/BranchProtection.cs

Api
https://docs.github.com/en/rest/branches/branch-protection#get-branch-protection

[timrogers]

@chris-smith-zocdoc Thanks for reaching out! Would you be open to create a PR to add these fields?

[chris-smith-zocdoc]

@timrogers potentially. Could you point me toward something that explains what type of testing / documentation is required for these new properties

[JonruAlveus]

@chris-smith-zocdoc It looks like there's some options for integration tests in RepositoryBranchesClientTests.TheGetBranchProtectionMethod where you can test that the properties are pulling back correctly. It doesn't look like there's any unit testing. Documentation is just summary tags on classes & properties.

[nickfloyd]

I added "up for grabs" just in case @chris-smith-zocdoc is not able to get to this - this would definitely be a great addition to provide more functionality to our community while we work on generative models.