Skip to content

perf(ci): FLOW-010 cut 1 — drop aprender-compute dev-dep on aprender-core (Refs #5803) - #6148

Closed
noahgift wants to merge 1 commit into
mainfrom
fix/5803-tuner-move-v2
Closed

noahgift wants to merge 1 commit into
mainfrom
fix/5803-tuner-move-v2

Conversation

@noahgift

Copy link
Copy Markdown
Member

Supersedes #6127: same single move commit with the seat-2 findings folded in, still one commit and one ticket.

  1. ml-tuner documented as a deprecated no-op (Cargo.toml still declares ml-tuner = []).
  2. check_tuner_test_move.sh reads CI fragments 295 and 298 (mutants S16, S17 turn it red); the false L6 label is gone.
  3. The unproven "proved in the PR body" claim is replaced by what is true: compared at move time, digests pin the bodies now.
  4. grep rc>=2 fails closed (mutant S18 isolates it: red with the guard, green without).

--self-test: 26 passed, 0 failed; live check green.

🤖 Generated with Claude Code

…core (Refs #5803)

Re-cut of #6127 with the seat-2 findings folded in: ml-tuner documented as a deprecated no-op; check_tuner_test_move.sh now requires CI fragments 295 and 298 (mutants S16/S17), fails closed on grep rc>=2 (S18), drops the unproven 'proved in the PR body' claim and the false L6 label.

PMAT-Ticket: PMAT-5803
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>

@noahgift noahgift left a comment

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seat 1 (aprender-w5018, non-author) @d7ae7120bfa070d4427ea0ac605cb88a77e7adb4 — NOT SEATED, scope check (c164) fails at seat time.

verdict: HOLD
patch-ids: 1cd74457a7d4

Scope: one commit, one ticket (PMAT-Ticket: PMAT-5803), diff matches the title (drop aprender-compute dev-dep on aprender-core, move tuner tests to aprender-core, LAB night guard, 295/298 cmd fragments). Merge-of-main only, 2 parents on the head.
Blocker: PMAT-5803 has no roadmap entry — no docs/roadmaps/entries/PMAT-5803.yaml and no 'id: PMAT-5803' in roadmap.yaml, on this head or on main — so pmat cannot read the ticket this PR claims.
Fix: add the entry (fragment + roadmap.yaml append), push as a new commit; I will re-seat bound to the new head. The code half I did not re-verify until scope is readable.

@github-actions

Copy link
Copy Markdown

§13.11 rung 1 — quorum shadow verdict

S13-SHADOW pr=6148 head=d7ae7120bfa070d4427ea0ac605cb88a77e7adb4 verdict=REFUSE class=Q1 arm_rc=1

Shadow mode: this records a verdict and merges nothing. A refusal
to arm is not a block (§13 adds zero rows to §7) — the pull request is
exactly as green as it was.

@noahgift

Copy link
Copy Markdown
Member Author

superseded by #6165

@noahgift

Copy link
Copy Markdown
Member Author

superseded by #6166

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant