Skip to content

GH-50115: [Dev] Adjust GitHub Copilot configuration for preliminary reviews#50117

Open
kou wants to merge 1 commit into
apache:mainfrom
kou:copilot
Open

GH-50115: [Dev] Adjust GitHub Copilot configuration for preliminary reviews#50117
kou wants to merge 1 commit into
apache:mainfrom
kou:copilot

Conversation

@kou
Copy link
Copy Markdown
Member

@kou kou commented Jun 7, 2026

Rationale for this change

We tried auto GitHub Copilot reviews recently and discussed our experience for them:
https://lists.apache.org/thread/thq0dz19shxbrjypb81q5ltx8h0w54ob

We want to tune auto GitHub Copilot reviews:

What changes are included in this PR?

  • Disable auto GitHub Copilot reviews for draft PR
  • Add code review instructions for GitHub Copilot
    • Base text is generated by Gemini 3 Flash Preview and I adjusted it

Are these changes tested?

No. We need to merge this to test this.

Are there any user-facing changes?

No.

@kou kou requested review from AlenkaF and alamb June 7, 2026 08:34
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jun 7, 2026

⚠️ GitHub issue #50115 has been automatically assigned in GitHub to PR creator.

Copy link
Copy Markdown
Member

@AlenkaF AlenkaF left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks good to me, thanks!

Copy link
Copy Markdown
Contributor

@alamb alamb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @kou -- this looks like an improvement to me

One other suggestion, would be to give contributors some more guidance on how to respond to reviews

For example, perhaps we could add a sentence to https://arrow.apache.org/docs/developers/overview.html#pull-request-and-review that says something like

In order to improve our code and your review experience, we have enabled
Github Copilot reviews to automate the first pass. Please address all copilot
comments by either implementing them, or explicitly responding and explaining
why you chose not to.

@github-actions github-actions Bot added awaiting merge Awaiting merge and removed awaiting committer review Awaiting committer review labels Jun 8, 2026
…nary reviews

Generated-by: Gemini 3 Flash Preview
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants