- Replaced with the suggested sentence.
- Replaced with: The suite leaves "Use Blockscout" off, so Blockscout is not asked. The rest of the bullet is unchanged.
Judgement call: the old sentence…
Ninth review (sneak/quak#228 (comment)): all four findings fixed.
- The
src/cli-log.tscomment now says every line quak logs goes to stderr, a command that…
Recorded (sneak in chat, 26 October): every path argument is resolved to its real path, with symlinks followed, before the scan begins. Then:
sfdupes scan /srv /srv/link/subresolves the…
Review: FAIL (needs-rework)
README.md, External Services, Blockscout entry, While the setting is off: "each of those screens shows a line saying what the screen leaves out and that…
Ninth review of sneak/quak#228 (sneak/quak#219): FAIL, needs rework. README.md needs no change. One finding is in a code comment, three…
Review failed: needs rework.
internal/handlers/webhook_stats_test.go,TestRelativeTime: the future-time case is exactly one hour ahead, whichrelativeTimewrites as0seven without its…
Eighth review of sneak/quak#228 (sneak/quak#219): FAIL, needs rework. README.md and the commit need no change. The three findings are in…
Rework for sneak/AutistMask#671 (comment):
src/popup/views/transactionDetail.js: comment replaced with your wording.- `tests/approvalUseBlockscoutOff.tes…
Seventh review of sneak/quak#228 (sneak/quak#219): FAIL, needs rework. The findings of the sixth review and the four further fixes are in…
Review: FAIL (needs-rework)
src/popup/views/transactionDetail.js, the comment inrender()over the calldata section: "Hide calldata and raw data sections; always fetch full tx…
FAIL: needs-rework. Line numbers refer to TODO.md at the PR head. All three sentences are in the sneak/sfdupes#125 entry.
- L159-160: "The rate a progress…
FAIL: needs-rework. Line numbers refer to README.md at the PR head.
- L352-353, "Operands": "No file is then reached twice." This is false. Two operands can reach one file under two paths…
The five findings of the sixth review are fixed with its wording: L161-163 ("The run fetches…"), L692 ("The JSON also holds … for the photo"), L860-861 (backup.lock and <dir> named),…