The summary line after sum, check and clear prints the unit twice, for example 9 B bytes or 1.0 KiB bytes. Stats.Print in attrsum.go puts bytes after the output of formatBytes, which already ends in a unit (B, KiB, MiB, ...). Found while working on #11.
Implementation
Drop the extra word from the format string in Stats.Print, so the line reads, for example, ... 3 failed, 1.0 KiB in 12ms.
A test of the summary line's text for one small and one large byte count.
TODO.md updated in the same change.
Definition of done
The test passes, and fails on the current code.
make check and script/cibuild pass. PR to next, independent review, squash.
Model: opus-5-5
The summary line after `sum`, `check` and `clear` prints the unit twice, for example `9 B bytes` or `1.0 KiB bytes`. `Stats.Print` in `attrsum.go` puts ` bytes` after the output of `formatBytes`, which already ends in a unit (`B`, `KiB`, `MiB`, ...). Found while working on https://git.eeqj.de/sneak/attrsum/issues/11.
## Implementation
- Drop the extra word from the format string in `Stats.Print`, so the line reads, for example, `... 3 failed, 1.0 KiB in 12ms`.
- A test of the summary line's text for one small and one large byte count.
- `TODO.md` updated in the same change.
## Definition of done
- The test passes, and fails on the current code.
- `make check` and `script/cibuild` pass. PR to `next`, independent review, squash.
Model: opus-5-5
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.
The summary line after
sum,checkandclearprints the unit twice, for example9 B bytesor1.0 KiB bytes.Stats.Printinattrsum.goputsbytesafter the output offormatBytes, which already ends in a unit (B,KiB,MiB, ...). Found while working on #11.Implementation
Stats.Print, so the line reads, for example,... 3 failed, 1.0 KiB in 12ms.TODO.mdupdated in the same change.Definition of done
make checkandscript/cibuildpass. PR tonext, independent review, squash.Model: opus-5-5