Fix the error that will encounter if 'Require pull request reviews before merging' is already enabled. (#12)

psyhomb · web-flow · commit 6aa6fefa4544 · 2020-05-06T20:22:23.000-04:00
2020/05/05 09:27:58 PUT https://api.github.com/repos/:owner/:repo/branches/:branch/protection: 422 Invalid request. No subschema in "anyOf" matched. 0 must be greater than or equal to 1. Not all subschemas of "allOf" matched. For 'anyOf/1', {"dismissal_restrictions"=>{"users"=>[], "teams"=>[]}, "dismiss_stale_reviews"=>false, "require_code_owner_reviews"=>false, "required_approving_review_count"=>0} is not a null. []
diff --git a/main.go b/main.go
@@ -181,6 +181,7 @@ func main() {
 			pullRequestReviewsEnforcementRequest.DismissalRestrictionsRequest = dismissalRestrictionsRequest
 			pullRequestReviewsEnforcementRequest.DismissStaleReviews = requiredPullRequestReviews.DismissStaleReviews
 			pullRequestReviewsEnforcementRequest.RequireCodeOwnerReviews = requiredPullRequestReviews.RequireCodeOwnerReviews
+			pullRequestReviewsEnforcementRequest.RequiredApprovingReviewCount = requiredPullRequestReviews.RequiredApprovingReviewCount
 			protectionRequest.RequiredPullRequestReviews = pullRequestReviewsEnforcementRequest
 		}
 

Original file line number	Diff line number	Diff line change
`@@ -181,6 +181,7 @@ func main() {`
`181`	`181`	`pullRequestReviewsEnforcementRequest.DismissalRestrictionsRequest = dismissalRestrictionsRequest`
`182`	`182`	`pullRequestReviewsEnforcementRequest.DismissStaleReviews = requiredPullRequestReviews.DismissStaleReviews`
`183`	`183`	`pullRequestReviewsEnforcementRequest.RequireCodeOwnerReviews = requiredPullRequestReviews.RequireCodeOwnerReviews`
	`184`	`+ pullRequestReviewsEnforcementRequest.RequiredApprovingReviewCount = requiredPullRequestReviews.RequiredApprovingReviewCount`
`184`	`185`	`protectionRequest.RequiredPullRequestReviews = pullRequestReviewsEnforcementRequest`
`185`	`186`	`}`
`186`	`187`