Skip to content

Commit 7bc3e9a

Browse files
Kasper JungeRalphify
authored andcommitted
workspace: record _plural f-string wrap cleanup
Co-authored-by: Ralphify <noreply@ralphify.co>
1 parent d34e957 commit 7bc3e9a

2 files changed

Lines changed: 7 additions & 1 deletion

File tree

workspace/ralphs/improve-codebase/coverage/_console_emitter.md

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,14 @@
11
# `_console_emitter.py` coverage
22

3-
Valid at: fc5e1cb
3+
Valid at: d34e957
44

55
## Recent changes
66

7+
- d34e957 — dropped redundant `f"{_plural(total, 'line')}"` wrap in
8+
`_FullscreenPeek._build_header`. `_plural` already returns a str
9+
so the f-string just format-identity-copied it. Same shape as the
10+
surrounding `header.append(literal, style=...)` calls. No other
11+
`f"{_plural(...)}"` wraps remain in the module (checked with grep).
712
- fc5e1cb — inlined `total_in = self._input_tokens` alias in
813
`_IterationPanel._format_tokens`. The rename hinted at a "total
914
input" aggregate that no longer exists (cache-read tokens are

workspace/ralphs/improve-codebase/iterations.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@
22

33
One line per iteration: `<sha> <summary>`.
44

5+
d34e957 refactor: drop redundant f-string wrap around `_plural(total, 'line')` in fullscreen header
56
fc5e1cb refactor: inline `total_in` alias in `_format_tokens` — direct read of `self._input_tokens` clarifies intent
67
3838006 refactor: defer `panel_for` guard to `is_live` to dedupe identical condition
78
01f2f1c refactor: drop `_reset_view` in `_FullscreenPeek` — identical body to `scroll_to_bottom`

0 commit comments

Comments
 (0)