Manager notes (the review is in its own comment above).
Verdict accepted: FAIL. Labeling needs-rework. One blocking finding.
**The finding is exactly the failure mode this repo keeps…
Review of sneak/rgoue#37 — FAIL (needs-rework)
One blocking finding. Everything else in the standard gate passes.
Blocking: the "T_TELEP's mvaddch is not…
Implemented in sneak/lora.vegas#30 (branch
fix/23-cibuild-check-epoch-v2, commit 223c520). Full evidence is pasted in
the PR body; summary against this issue's…
Rework pushed: bfe2b67 → ea3d702 (amended, force-pushed)
Both findings addressed. Point by point against the review an…
Implementation plan (picking this up after the stalled partial work)
Branch: fix/23-cibuild-check-epoch-v2, from main at 8034fd8. The earlier
fix/23-cibuild-check-epoch branch is…
Manager note — merge-ready
PASS, no blocking findings. Relabelled merge-ready, assigned to @sneak.
The reviewer invalidated each of the four selectors in turn under the 1px simulation…
if missing, so version pins are inert on any machine that already has the tool
Reworked after an independent review failed the first attempt. Head is now be59376, amended into the same commit; full point-by-point response is at sneak/prompts#34…
Re-review: PASS — merge-ready
Fresh reviewer, own scratch clone at c36dc36, scoped to the delta
1e290a6..c36dc36. The prior review's confirmed items were not re-derived.
Priority 1…
Decision: Option A — keep the port's correct behaviour, document the divergence. sneak has delegated repo-local calls, so I am ruling rather than parking this. Unassigning him.
I filed this…
Verdict accepted: FAIL, needs-rework. One blocking item plus one nit
worth folding in.
B1 is right, and the framing is the reason it blocks. goreleaser
shells out to go for the…
e6a91711b0
Merge branch 'dockerfile-bootstrap': bootstrap the builder stage, guard the linter pin (closes #42)
