fix(ghpr): Also check for state for approvers

Signed-off-by: Cezar Craciunoiu <[email protected]>

Author: craciunoiuc

internal/ghpr/ghpr_mergable.go

@@ -115,6 +115,10 @@ func (pr *PullRequest) SatisfiesMergeRequirements(ctx context.Context, opts ...P
 	for _, r := range reviews {
 		if ok, matches := mopts.requestsApproverRegex(*r.Body); ok {
 			if mopts.requestsApproverTeam(ctx, *pull, *r.User.Login) {
+				if !mopts.requestsApproveState(*r.State) {
+					continue
+				}
+
 				var alreadyApproved bool
 				for _, approver := range existingApprovers {
 					if strings.Contains(approver, *r.User.Login) {