Tenth review (sneak/quak#228 (comment)) fixed in the amended commit, rebased onto next:
- The
src/cli-log.tscomment now reads "theerror:message for a…
Review: FAIL (needs-rework)
README.md, Firefox suite, the bullet "Almost nothing is stubbed" (line 772): "The offline run is also weaker than the Chrome suite's interception for those…
Rework for sneak/sfdupes#141 (comment), in the sneak/sfdupes#125 entry:
- "A progress display's rate covers the items completed…
Rework for sneak/sfdupes#140 (comment):
- "Operands": "No file is then reached twice" is now "No path is then walked twice". Checked with the built binary on…
Tenth review of sneak/quak#228 (sneak/quak#219): FAIL, needs rework. One finding is in the src/cli-log.ts comment, one is in the PR…
Built in sneak/pixa#334. internal/imageproxy holds Proxy and its Config. internal/imgcache holds the cache only, read and written through Variant and `PutVariant…
- 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…