Merge pull request #41 from capthiron/fix/null-check-for-commit-status

fix(check-status): add null check for commit status rollup
github · Nov 1, 2023 · efe4892 · efe4892
2 parents 3dcd81a + 4e299dc
commit efe4892
Showing 1 changed file with 9 additions and 4 deletions.
diff --git a/src/functions/check-status.js b/src/functions/check-status.js
@@ -37,10 +37,15 @@ export async function checkStatus(
     // Check for CI status
     if (mustBeGreen) {
       const [{commit}] = result.repository.pullRequest.commits.nodes
-      const state = commit.statusCheckRollup.state
-      core.info('Validating status: ' + state)
-      if (state !== 'SUCCESS') {
-        core.info('Discarding ' + branch + ' with status ' + state)
+      if (commit.statusCheckRollup) {
+        const state = commit.statusCheckRollup.state
+        core.info('Validating status: ' + state)
+        if (state !== 'SUCCESS') {
+          core.info('Discarding ' + branch + ' with status ' + state)
+          statusOK = false
+        }
+      } else {
+        core.info('No status check associated with branch: ' + branch)
         statusOK = false
       }
     }