Skip to content

ci: enable Copilot draft reviews and skip unrelated checks - #886

Open
manuzhang wants to merge 3 commits into
apache:mainfrom
manuzhang:agent/enable-copilot-draft-reviews
Open

ci: enable Copilot draft reviews and skip unrelated checks#886
manuzhang wants to merge 3 commits into
apache:mainfrom
manuzhang:agent/enable-copilot-draft-reviews

Conversation

@manuzhang

@manuzhang manuzhang commented Aug 12, 2026

Copy link
Copy Markdown
Member

Rationale for this change

ASF-managed Copilot code review is enabled for this repository, but automatic reviews are currently disabled while pull requests are drafts. Enabling draft reviews provides earlier feedback before human review is requested and complements the existing review-on-push setting.

A pull request changing only .asf.yaml also starts unrelated build and test workflows. Those workflows do not validate ASF repository configuration and consume unnecessary CI resources.

What changes are included in this PR?

  • Enable automatic Copilot code review for draft pull requests in .asf.yaml
  • Keep automatic Copilot review on new pushes enabled
  • Ignore .asf.yaml-only pull requests in the sanitizer, AWS, C++ linter, SQL catalog, and general test workflows
  • Keep the lightweight pre-commit and license workflows enabled for .asf.yaml changes
  • Leave main branch and tag workflow triggers unchanged

Are these changes tested?

Yes. .asf.yaml and all modified workflow files were parsed successfully as YAML, the configured review_drafts value was verified as boolean true, and git diff --check passed.

Are there any user-facing changes?

No.

Co-authored-by: Codex <codex@openai.com>
@manuzhang
manuzhang marked this pull request as ready for review August 12, 2026 14:03
@manuzhang
manuzhang requested review from wgtmac and a lite review from Copilot August 12, 2026 14:03

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Enables ASF-managed GitHub Copilot code review to run automatically on draft pull requests, allowing earlier automated feedback while preserving the existing review-on-push behavior.

Changes:

  • Set github.copilot_code_review.review_drafts to true in .asf.yaml
  • Leave github.copilot_code_review.review_on_push enabled

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Co-authored-by: Codex <codex@openai.com>
Copilot AI review requested due to automatic review settings August 12, 2026 14:08

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 7 out of 7 changed files in this pull request and generated no new comments.

Co-authored-by: Codex <codex@openai.com>
Copilot AI review requested due to automatic review settings August 12, 2026 14:49
@manuzhang manuzhang changed the title ci: enable Copilot reviews for draft PRs ci: enable Copilot draft reviews and skip unrelated checks Aug 12, 2026

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 6 out of 6 changed files in this pull request and generated no new comments.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants