pub enum WaitOutcome {
Passed,
Failed,
Inconclusive,
Landed,
}Expand description
The result of waiting on a review’s checks before merging it.
Variants§
Passed
Checks passed, or there are none - go ahead and merge.
Failed
A required check failed - stop the run.
Inconclusive
Checks stopped without a verdict - a cancelled run, or one waiting on a person. Nothing failed, but nothing passed either, and the platform still holds the merge, so the run stops and says which it was.
The same distinction CheckStatus::Inconclusive draws for the dot,
and deliberately so: a provider decides both, and the two must agree
about one commit or the gate contradicts the dot the user just read.
Where a provider’s gate cannot see the difference on its own - GitHub’s
reads gh pr checks exit codes, which have no code for it - it asks
the same rollup the dot came from.
Landed
The review merged out-of-band while we waited (an admin merge on the
web, say). Skip the redundant merge and let sync reconcile it.