huskies: merge 972
This commit is contained in:
@@ -340,6 +340,8 @@ fn map_stage_move_to_event(
|
|||||||
(Stage::MergeFailure { .. }, "backlog") => Ok(PipelineEvent::Demote),
|
(Stage::MergeFailure { .. }, "backlog") => Ok(PipelineEvent::Demote),
|
||||||
// Story 971: send MergeFailure story back to Coding so a coder can fix it.
|
// Story 971: send MergeFailure story back to Coding so a coder can fix it.
|
||||||
(Stage::MergeFailure { .. }, "current") => Ok(PipelineEvent::FixupRequested),
|
(Stage::MergeFailure { .. }, "current") => Ok(PipelineEvent::FixupRequested),
|
||||||
|
// Story 972: send MergeFailure story back to Qa for a QA agent to re-review.
|
||||||
|
(Stage::MergeFailure { .. }, "qa") => Ok(PipelineEvent::ReQueuedForQa),
|
||||||
(
|
(
|
||||||
Stage::Archived {
|
Stage::Archived {
|
||||||
reason: ArchiveReason::Blocked { .. },
|
reason: ArchiveReason::Blocked { .. },
|
||||||
|
|||||||
@@ -70,6 +70,8 @@ pub enum PipelineEvent {
|
|||||||
MergemasterAttempted,
|
MergemasterAttempted,
|
||||||
/// Story 971: user sends a MergeFailure story back to Coding for coder fixup.
|
/// Story 971: user sends a MergeFailure story back to Coding for coder fixup.
|
||||||
FixupRequested,
|
FixupRequested,
|
||||||
|
/// Story 972: user sends a MergeFailure story back to Qa for re-review.
|
||||||
|
ReQueuedForQa,
|
||||||
}
|
}
|
||||||
|
|
||||||
// ── Per-node execution events ───────────────────────────────────────────────
|
// ── Per-node execution events ───────────────────────────────────────────────
|
||||||
@@ -114,6 +116,7 @@ pub fn event_label(e: &PipelineEvent) -> &'static str {
|
|||||||
PipelineEvent::ReviewHoldCleared => "ReviewHoldCleared",
|
PipelineEvent::ReviewHoldCleared => "ReviewHoldCleared",
|
||||||
PipelineEvent::MergemasterAttempted => "MergemasterAttempted",
|
PipelineEvent::MergemasterAttempted => "MergemasterAttempted",
|
||||||
PipelineEvent::FixupRequested => "FixupRequested",
|
PipelineEvent::FixupRequested => "FixupRequested",
|
||||||
|
PipelineEvent::ReQueuedForQa => "ReQueuedForQa",
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
@@ -304,6 +307,9 @@ pub fn transition(state: Stage, event: PipelineEvent) -> Result<Stage, Transitio
|
|||||||
// ── FixupRequested: MergeFailure → Coding (coder fixup) ────────
|
// ── FixupRequested: MergeFailure → Coding (coder fixup) ────────
|
||||||
(MergeFailure { .. }, FixupRequested) => Ok(Coding),
|
(MergeFailure { .. }, FixupRequested) => Ok(Coding),
|
||||||
|
|
||||||
|
// ── ReQueuedForQa: MergeFailure → Qa (re-review) ────────────────
|
||||||
|
(MergeFailure { .. }, ReQueuedForQa) => Ok(Qa),
|
||||||
|
|
||||||
// ── MergemasterAttempted: MergeFailure → MergeFailureFinal ─────
|
// ── MergemasterAttempted: MergeFailure → MergeFailureFinal ─────
|
||||||
(MergeFailure { reason, .. }, MergemasterAttempted) => Ok(MergeFailureFinal { reason }),
|
(MergeFailure { reason, .. }, MergemasterAttempted) => Ok(MergeFailureFinal { reason }),
|
||||||
(MergeFailureFinal { reason }, MergemasterAttempted) => Ok(MergeFailureFinal { reason }),
|
(MergeFailureFinal { reason }, MergemasterAttempted) => Ok(MergeFailureFinal { reason }),
|
||||||
|
|||||||
Reference in New Issue
Block a user