Skip to content

Contradictory state in configuring copilot code review #37641

Open
@mattantaliss

Description

@mattantaliss

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

Metadata

Metadata

Assignees

No one assigned

    Labels

    contentThis issue or pull request belongs to the Docs Content teamcopilotContent related to GitHub Copilot

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions