Commit 6464b2a
Julian Knutsen
review fixups: wake-budget test coverage, drop cli.md churn, docstring
Addresses adoption-review findings for PR #818:
- Add table test TestDaemonConfig_MaxWakesPerTickOrDefault covering
nil / zero / negative / positive branches plus TestParseMaxWakesPerTick
for the TOML round-trip. Pins the <=0 fallback guarantee the PR
description relies on.
- Drop ~516 lines of docs/reference/cli.md reformatting churn from this
PR. That was genschema table-formatting churn unrelated to the
wake-budget change; it belongs in a separate chore(docs) PR.
- Extend MaxWakesPerTick docstring to name the thundering-herd tradeoff
(restart burst vs steady-state throughput) that the original constant
was chosen to bound.1 parent f563666 commit 6464b2a
3 files changed
Lines changed: 58 additions & 516 deletions
0 commit comments