Skip to content

Commit 3ba818e

Browse files
author
Jiayang Zhou
committed
feat: add copilot_code_review rule support to repository rulesets
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
1 parent 910c0ea commit 3ba818e

File tree

2 files changed

+15
-15
lines changed

2 files changed

+15
-15
lines changed

github/rules.go

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -981,14 +981,14 @@ func (r *RepositoryRulesetRules) UnmarshalJSON(data []byte) error {
981981
return err
982982
}
983983
}
984-
case RulesetRuleTypeCopilotCodeReview:
985-
r.CopilotCodeReview = &CopilotCodeReviewRuleParameters{}
984+
case RulesetRuleTypeCopilotCodeReview:
985+
r.CopilotCodeReview = &CopilotCodeReviewRuleParameters{}
986986

987-
if w.Parameters != nil {
988-
if err := json.Unmarshal(w.Parameters, r.CopilotCodeReview); err != nil {
989-
return err
990-
}
991-
}
987+
if w.Parameters != nil {
988+
if err := json.Unmarshal(w.Parameters, r.CopilotCodeReview); err != nil {
989+
return err
990+
}
991+
}
992992
case RulesetRuleTypeRepositoryCreate:
993993
r.RepositoryCreate = &EmptyRuleParameters{}
994994
case RulesetRuleTypeRepositoryDelete:

github/rules_test.go

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -122,10 +122,10 @@ func TestRulesetRules(t *testing.T) {
122122
},
123123
},
124124
},
125-
CopilotCodeReview: &CopilotCodeReviewRuleParameters{
126-
ReviewNewPushes: true,
127-
ReviewDraftPullRequests: false,
128-
},
125+
CopilotCodeReview: &CopilotCodeReviewRuleParameters{
126+
ReviewNewPushes: true,
127+
ReviewDraftPullRequests: false,
128+
},
129129
RepositoryCreate: &EmptyRuleParameters{},
130130
RepositoryDelete: &EmptyRuleParameters{},
131131
RepositoryName: &SimplePatternRuleParameters{Pattern: "^test-.+", Negate: false},
@@ -244,10 +244,10 @@ ReviewDraftPullRequests: false,
244244
},
245245
},
246246
},
247-
CopilotCodeReview: &CopilotCodeReviewRuleParameters{
248-
ReviewNewPushes: true,
249-
ReviewDraftPullRequests: false,
250-
},
247+
CopilotCodeReview: &CopilotCodeReviewRuleParameters{
248+
ReviewNewPushes: true,
249+
ReviewDraftPullRequests: false,
250+
},
251251
RepositoryCreate: &EmptyRuleParameters{},
252252
RepositoryDelete: &EmptyRuleParameters{},
253253
RepositoryName: &SimplePatternRuleParameters{Pattern: "^test-.+", Negate: false},

0 commit comments

Comments
 (0)