[Fix] not all commits have checks on them

ljharb · Mar 14, 2022 · 0b8b138 · 0b8b138
1 parent 89021b3
commit 0b8b138
Showing 1 changed file with 9 additions and 7 deletions.
diff --git a/utils/evaluateChecks.js b/utils/evaluateChecks.js
@@ -45,13 +45,15 @@ function parseCheckRuns(statusCheckRollup, requiredChecks) {
 module.exports = function evaluateChecks(pullRequest, requiredChecks) {
 	const { commits: { nodes: [{ commit: { statusCheckRollup } }] } } = pullRequest;
 
-	for (const ctx of statusCheckRollup.contexts.nodes) {
-		/* eslint no-underscore-dangle: 0 */
-		if (ctx.__typename === 'StatusContext' && ctx.state !== 'SUCCESS') {
-			return parseStatusChecks(statusCheckRollup, requiredChecks);
-		}
-		if (ctx.__typename === 'CheckRun' && ctx.conclusion !== 'SUCCESS') {
-			return parseCheckRuns(statusCheckRollup, requiredChecks);
+	if (statusCheckRollup?.context?.nodes) {
+		for (const ctx of statusCheckRollup.contexts.nodes) {
+			/* eslint no-underscore-dangle: 0 */
+			if (ctx.__typename === 'StatusContext' && ctx.state !== 'SUCCESS') {
+				return parseStatusChecks(statusCheckRollup, requiredChecks);
+			}
+			if (ctx.__typename === 'CheckRun' && ctx.conclusion !== 'SUCCESS') {
+				return parseCheckRuns(statusCheckRollup, requiredChecks);
+			}
 		}
 	}