Skip to content

Commit 9099120

Browse files
authored
Exclude "Automated merging requirements met" when getting list of required checks (#36513)
1 parent b5560b7 commit 9099120

File tree

1 file changed

+6
-1
lines changed

1 file changed

+6
-1
lines changed

.github/workflows/src/summarize-checks/summarize-checks.js

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -785,7 +785,12 @@ export async function getCheckRunTuple(
785785
});
786786

787787
if (branchRules) {
788-
requiredCheckNames = getRequiredChecksFromBranchRuleOutput(branchRules);
788+
requiredCheckNames = getRequiredChecksFromBranchRuleOutput(branchRules).filter(
789+
// "Automated merging requirements met" may be required in repo settings, to ensure PRs cannot be merged unless
790+
// it's passing. However, it must be excluded from our list of requiredCheckNames, since it's status is set
791+
// by our own workflow. If this check isn't excluded, it creates a deadlock where it can never be set.
792+
(checkName) => checkName !== AUTOMATED_CHECK_NAME,
793+
);
789794
}
790795
} else {
791796
requiredCheckNames = ["Summarize PR Impact", "[TEST-IGNORE] Summarize PR Impact"];

0 commit comments

Comments
 (0)