Skip to content

Commit

Permalink
fix: treat renovate/stability-days pending status as failed to avoid …
Browse files Browse the repository at this point in the history
…blocking prs that can be merged
  • Loading branch information
christophehurpeau committed Jan 2, 2022
1 parent 0e8d7a4 commit ca79c7e
Showing 1 changed file with 9 additions and 0 deletions.
9 changes: 9 additions & 0 deletions src/events/pr-handlers/actions/autoMergeIfPossible.ts
Expand Up @@ -348,6 +348,15 @@ export const autoMergeIfPossible = async <
} else if (pendingChecks.length > 0 || pendingStatuses.length > 0) {
addLog('pending status or checks', 'remove');
await createAutomergeStatus('pending statuses or checks');
if (pendingStatuses.includes('renovate/stability-days')) {
// renovate won't likely change this status in the next minutes
await repoContext.removePrFromAutomergeQueue(
context,
pullRequest,
'failed status or checks',
);
return false;
}
// waiting for reschedule in status (pr-handler/status.ts), or retry anyway and if it fails remove from queue
await repoContext.reschedule(context, createMergeLockPrFromPr(), true);
return false;
Expand Down

0 comments on commit ca79c7e

Please sign in to comment.