Contradictory state in configuring copilot code review

[mattantaliss]

Code of Conduct

• I have read and agree to the GitHub Docs project's Code of Conduct

What article on docs.github.com is affected?

https://docs.github.com/en/copilot/using-github-copilot/code-review/configuring-automatic-code-review-by-copilot

What part(s) of the article would you like to see updated?

In configuring automatic code review for a single repository, one of the suggested options when creating a target branch is to select 'Include all branches'. However, to activate automatic code review, the 'Require a pull request before merging' checkbox needs to be enabled, which requires any commit made to a non-target branch be made via a pull request. When the branch targeting is 'all branches', there is no non-target branch to get a commit onto in order to create the required PR. In that state, no commits can be pushed to any branch.

I believe " or Include all branches" should be deleted from that target branch step to avoid a contradictory state.

Additional information

No response

[Sharra-writes]

Thanks so much for opening an issue! I'll get this triaged for review.

[rancerryan]

Thankss

[Dickywahyudi771]

const array = [1, 2, 3, 4, 5];

[Sharra-writes]

@mattantaliss I've been looking into this, and it seems if there's a problem it's probably with the the product rather than the documentation. Someone in the OS community might be able to help you figure out if you're misunderstanding something, but if it is a bug, the support team is probably your best bet.