Scope addition, from the PR #34 review: the sweep is not three files per repo. It is a grep.
The canonical fix passed code review and failed on scope — five sites across three org-canonic…
Reviewer note: the one check that is easy to get wrong is item 4. docker run --user 0:0 on the builder target returns ok ... (cached) and exits 0 unless the Go test cache is disabled; with…
Correction: the overflow assertion I specified would have false-passed
I wrote document.documentElement.scrollWidth <= innerWidth into the definition of done. That check **passes on a…
Implemented in PR #34 (branch test/rings-coverage, commit c61e282):
sneak/rgoue#34
New game/rings_test.go — 17 tests, 44 subtests, no game-code changes —
covering…
What is covered
game/rings_test.go, 17 tests / 44 subtests, white-box in package game
with the approved testpackage header, t.Parallel() on every test and
every subtest. No game code…
Implemented in sneak/dnswatcher#131 (branch fix/117-bootstrap-pin, commit db933f3).
What changed in script/bootstrap
golangci-lintandgoimportsare…
Addendum to the verification above, because the cache-collection item (#99.4) failed its own negative control before it passed one, and the failure is worth recording rather than quietly…
Done in #44. Harness built, harness proven able to fail (twice — planted
wide element, and the reflow rule neutered), limits documented in
test/viewport/README.md so this does not get re-parked…
Verbatim output of make frontend-viewport-test on this branch, so a
reviewer can diff their own run against it. Rationale and the
harness-can-fail proof are in the PR body; this is just the…