aboutsummaryrefslogtreecommitdiff
path: root/cmd/krino/review_test.go
diff options
context:
space:
mode:
authorLukasz Kasprzak <lukas@labunix.xyz>2026-09-17 12:11:42 +0200
committerLukasz Kasprzak <lukas@labunix.xyz>2026-09-17 12:11:42 +0200
commitbddbd74e4a73e8e32bcf648efd1cac5655f6d0cd (patch)
tree187e6a1fb722d9ab97d2076f69f997f5d371e943 /cmd/krino/review_test.go
parentcd7425b81f963a948f0abe7df3f9e58e190c2b78 (diff)
downloadkrino-bddbd74e4a73e8e32bcf648efd1cac5655f6d0cd.tar.gz
krino-bddbd74e4a73e8e32bcf648efd1cac5655f6d0cd.zip
comments that explain the code, not how it was written
About 340 comments cited the development process: task and plan numbers, fix waves, rulings, reviewers, and the author in the third person with a date. None of that exists outside the work itself, so to a reader it pointed at nothing. Each one now states the engineering reason it was standing in front of; where a comment was provenance and nothing else, it is gone. References to docs/design.md and docs/gui-design.md by section stay: both ship with the repository. The design documents lose their amendment diaries - CHANGELOG.md is that record - and the GUI's says plainly that the window has gone further than the document. Only comments changed. Every .go file was parsed and its code printed with comments stripped, before and after: the two hashes are identical across all 175 files.
Diffstat (limited to 'cmd/krino/review_test.go')
-rw-r--r--cmd/krino/review_test.go12
1 files changed, 6 insertions, 6 deletions
diff --git a/cmd/krino/review_test.go b/cmd/krino/review_test.go
index fc175cb..0731ba9 100644
--- a/cmd/krino/review_test.go
+++ b/cmd/krino/review_test.go
@@ -111,11 +111,11 @@ func TestInvalidKeyReprompts(t *testing.T) {
}
}
-// TestPerFileDestinationIsRootRelative is review finding 1 (fix round
-// 2026-09-12): reviewPerFile must render a destination the same way the
-// directory-level table does (render.go's destText) - root-relative for a
-// destination inside root, ~-abbreviated for one outside it - not always
-// abbreviated because root was never passed through to actionCell at all.
+// TestPerFileDestinationIsRootRelative: reviewPerFile must render a
+// destination the same way the directory-level table does (render.go's
+// destText) - root-relative for a destination inside root, ~-abbreviated
+// for one outside it - not always abbreviated because root was never
+// passed through to actionCell at all.
// Both halves are pinned: getting only the "inside" half right would still
// let an outside-root destination silently regress to some other form.
func TestPerFileDestinationIsRootRelative(t *testing.T) {
@@ -330,7 +330,7 @@ func TestStopAfterApply(t *testing.T) {
// TestReviewHeadingsCannotFakeAStepLine: a long name in the per-file
// heading of review and of undo wraps with its continuation past the column
-// step labels use, so the name cannot pass for a step (plan 11 review L4).
+// step labels use, so the name cannot pass for a step.
func TestReviewHeadingsCannotFakeAStepLine(t *testing.T) {
old := widthPolicy
t.Cleanup(func() { widthPolicy = old })