Skip to content

Commit 865919b

Browse files
committed
docs(plan/2606231013): correct stale task 4 description
Task 4 said to add an exemption comment for isSpaceOrNewlineByte. In practice we added TestIsSpaceOrNewlineByte because the predicate has two boolean branches and the trivial-accessor exemption doesn't apply. Update the task text to match what was done. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01GYs45JNFoMgto6dk5mSSXb
1 parent a9c9135 commit 865919b

1 file changed

Lines changed: 4 additions & 4 deletions

File tree

plan/2606231013_arch-fix-inline-scan-helper-tests.md

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -59,10 +59,10 @@ test as of commit 1599c9f:
5959
3. Keep each test function short (≤ 15 lines).
6060
Use table-driven style only when three or
6161
more cases differ in one dimension.
62-
4. `isSpaceOrNewlineByte` is a one-liner with
63-
no branch; add the one-line "no test by design"
64-
exemption comment in `inline_scan.go` instead
65-
of a test.
62+
4. `isSpaceOrNewlineByte` has two boolean branches
63+
(`c == ' '` and `c == '\n'`), so the trivial-
64+
accessor exemption does not apply. Add
65+
`TestIsSpaceOrNewlineByte` instead.
6666
5. Run `go test ./internal/lint/... -run TestScan`
6767
to confirm all pass.
6868
6. Run `go vet ./internal/lint/...` and

0 commit comments

Comments
 (0)