Skip to content

Spec 001: review notes' slips - #177

Closed
adamjohnwright wants to merge 1 commit into
mainfrom
docs/review-slips
Closed

adamjohnwright wants to merge 1 commit into
mainfrom
docs/review-slips

Conversation

@adamjohnwright

Copy link
Copy Markdown
Contributor

Spec 001, notes and comments only, from the lows of #174, #175 and #176's reviews:

  • T035's note nested as it was before Spec 001 T035 follow-up: pdf.js's error never printed whole #175 (its follow-up after Spec 001 T035: compare #171's review; the gaps for T040 under the done note again).
  • The load flakes' note and the breaker test's comment give the margins as measured (0.25 s and 0.5 s, not 0.8 s or 0.45 s).
  • T047's note lists all eleven folder tests.
  • The book signal tests' signalled comment describes the poll as it now works (sleep until the pages' folder, then without pause).

No code changes.

🤖 Generated with Claude Code

T035's note nested as it was (#175's follow-up after #171's review, the gaps
under the done note); the load flakes' and the breaker's margins as
measured; T047's folder tests listed in full; the signal tests' poll
described as it now works. Comments and notes only.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@adamjohnwright

Copy link
Copy Markdown
Contributor Author

Replaced after review: two of its wording fixes were themselves wrong.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant