[Fix] display review status first as output now shows check status

ljharb · Mar 14, 2022 · 7f6cdb7 · 7f6cdb7
1 parent a8a1fb3
commit 7f6cdb7
Showing 1 changed file with 7 additions and 6 deletions.
diff --git a/utils/evaluatePullRequest.js b/utils/evaluatePullRequest.js
@@ -24,6 +24,13 @@ module.exports = function evaluatePullRequest(response, requiredChecks) {
 	if (!doesPRHaveConflicts(response)) {
 		return pullRequestStatus.CONFLICT;
 	}
+
+	if (reviewDecision === 'CHANGES_REQUESTED') {
+		return pullRequestStatus.REVIEW_DISAPPROVED;
+	} else if (reviewDecision === 'REVIEW_REQUIRED') {
+		return pullRequestStatus.REVIEW_REQUIRED;
+	}
+
 	const { failure, pending } = evaluateChecks(response, requiredChecks);
 
 	if (failure.some((f) => f.isRequired)) {
@@ -34,11 +41,5 @@ module.exports = function evaluatePullRequest(response, requiredChecks) {
 		return pullRequestStatus.STATUS_PENDING;
 	}
 
-	if (reviewDecision === 'CHANGES_REQUESTED') {
-		return pullRequestStatus.REVIEW_DISAPPROVED;
-	} else if (reviewDecision === 'REVIEW_REQUIRED') {
-		return pullRequestStatus.REVIEW_REQUIRED;
-	}
-
 	return pullRequestStatus.MERGEABLE;
 };