Repository navigation
linux: work-598 - #958
linux: work-598#958
Conversation
Agent-Team: linux Worker: codex
PM note: how this PR will be handledWhat it is for: linux: work-598 Rules that apply: none special. Things to watch (checked by rule):
What happens next:
A living status comment below is kept current. Nothing here is a merge decision. |
Where this PR standsStage: Done: approved What the PM needs before approving (ticks as it is met; the full CI runs only at the end, after the reviews are clean):
Reviews at the current commit (
Latest fixes comment: https://github.com/fizzyedit/work/issues/602#issuecomment-6463719041 (2 findings answered) Updated 2026-10-28 10:07 UTC by the PM. This comment is edited in place. |
Part of fizzyedit/work#598
Goal
Part of fizzyedit/work#598: follow up PR #914 by using the current standard library formatter.
What changed and why
Replaced
std.fmt.bufPrintwithstd.mem.printfor_input_time_{d}in SDLBackend and WebBackend, following docs/ZIG.md rule 8. No other source changes.How it was verified
Formatting, lint, native build, and diff whitespace checks passed. Unit tests: 888/915 passed, 3 skipped, 24 failed in socket-dependent suites under sandbox restrictions; names are recorded in GATES.md. Confirmed the installed std.mem.print signature. No new regression test for this API-only substitution; no UI behavior changes.
Not done / follow-ups
Harness must rerun the socket-dependent tests outside the sandbox and handle commit, push, and PR creation.
Changes (2 files, +2/-2)
Gates: 6 of 6 passed
scripts/check-fmt.sh(exit 0)scripts/zb build --summary all(exit 0)scripts/zb build test --summary all(exit 0)scripts/zb build test-integration --summary all(exit 0)scripts/zb build check-web --summary all(exit 0)scripts/zb build test-sdk-version --summary all(exit 0)