Skip to content

Contradictory state in configuring copilot code review #37641

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
1 task done
mattantaliss opened this issue Apr 18, 2025 · 1 comment
Open
1 task done

Contradictory state in configuring copilot code review #37641

mattantaliss opened this issue Apr 18, 2025 · 1 comment
Labels
content This issue or pull request belongs to the Docs Content team copilot Content related to GitHub Copilot

Comments

@mattantaliss
Copy link

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

@mattantaliss mattantaliss added the content This issue or pull request belongs to the Docs Content team label Apr 18, 2025
@github-actions github-actions bot added the triage Do not begin working on this issue until triaged by the team label Apr 18, 2025
@Sharra-writes
Copy link
Contributor

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

@Sharra-writes Sharra-writes added copilot Content related to GitHub Copilot and removed triage Do not begin working on this issue until triaged by the team labels Apr 18, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
content This issue or pull request belongs to the Docs Content team copilot Content related to GitHub Copilot
Projects
None yet
Development

No branches or pull requests

2 participants