Skip to content

Conversation

@JiayangZhou
Copy link

@JiayangZhou JiayangZhou commented Dec 4, 2025

Add CopilotCodeReviewRuleParameters struct with ReviewNewPushes and ReviewDraftPullRequests fields to support GitHub's Copilot code review rule for repository rulesets.

Changes:

  • Add RulesetRuleTypeCopilotCodeReview constant
  • Add CopilotCodeReviewRuleParameters struct
  • Add CopilotCodeReview field to RepositoryRulesetRules
  • Implement marshaling and unmarshaling logic
  • Add test coverage for the new rule type

this feature is based on https://github.blog/changelog/2025-09-10-copilot-code-review-independent-repository-rule-for-automatic-reviews/ (an independent repository rule instead of pull request rule)

@google-cla
Copy link

google-cla bot commented Dec 4, 2025

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

@JiayangZhou
Copy link
Author

This is for Github Terraform provider to support code review bot feature in repo ruleset, relevant PR:

integrations/terraform-provider-github#2965

@codecov
Copy link

codecov bot commented Dec 4, 2025

Codecov Report

❌ Patch coverage is 47.05882% with 9 lines in your changes missing coverage. Please review.
✅ Project coverage is 92.29%. Comparing base (66f826c) to head (72a87ba).

Files with missing lines Patch % Lines
github/rules.go 47.05% 7 Missing and 2 partials ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #3857      +/-   ##
==========================================
- Coverage   92.36%   92.29%   -0.07%     
==========================================
  Files         199      199              
  Lines       14308    14324      +16     
==========================================
+ Hits        13215    13221       +6     
- Misses        894      902       +8     
- Partials      199      201       +2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Add support for GitHub's Copilot code review feature in repository
rulesets, allowing automatic code review requests for pull requests.

Changes:
- Add RulesetRuleTypeCopilotCodeReview constant to rule types
- Add CopilotCodeReviewRuleParameters struct with review_new_pushes
  and review_draft_pull_requests fields
- Implement marshal/unmarshal logic for copilot_code_review rules
- Add comprehensive test coverage for the new rule type

The copilot_code_review rule supports two configuration options:
- review_new_pushes: Automatically review each new push to the PR
- review_draft_pull_requests: Automatically review draft PRs
@JiayangZhou JiayangZhou force-pushed the feature/copilot-code-review-ruleset branch from fab8541 to 2b90cfa Compare December 4, 2025 18:51
@gmlewis
Copy link
Collaborator

gmlewis commented Dec 4, 2025

Force-pushing is completely unnecessary. Please see CONTRIBUTING.md for why.

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