ReviewOS

also looking at this

stacks/buddy

fix: report skipped gates as skipped, and honour ai.majorUpgrades.maxAttempts

#1433
Merged glennmichael123 wants to merge fix/gate-neutral into main
9 files +257 -8
bin/cli.tsmodified+2-2
Changes to bin/cli.ts
@@ -3029,7 +3029,7 @@ cli
30293029 }
30303030
30313031 const provider = await providerFor(config, 'running gates', logger)
3032 const { findLinkedIssues, runAiGates, runGates, summarizeGates } = await import('../src/gates')
3032 const { findLinkedIssues, runAiGates, runGates, gateIcon, summarizeGates } = await import('../src/gates')
30333033 const { parseManifest } = await import('../src/pr/pr-manifest')
30343034
30353035 const prs = await provider.getPullRequests('open')
@@ -3105,7 +3105,7 @@ cli
31053105 const summary = summarizeGates(results)
31063106
31073107 for (const result of results)
3108 logger.info(`${result.passed ? '✅' : result.mode === 'error' ? '❌' : '⚠️'} ${result.name}: ${result.summary}`)
3108 logger.info(`${gateIcon(result)} ${result.name}: ${result.summary}`)
31093109
31103110 const blocking = summary.conclusion === 'failure'
31113111