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),…
Reworked against sneak/sfdupes#141 (comment): one amended commit, rebased onto current next.
- The sneak/sfdupes#106 entry: "With…
Reworked against sneak/sfdupes#140 (comment): one amended commit, rebased onto current next, which now holds sneak/sfdupes#145.
1.…
Sixth review of sneak/quak#228 (sneak/quak#219): FAIL, needs rework. The three findings of the fifth review are fixed. Line numbers are…
Rework (round 16), for sneak/AutistMask#671 (comment):
- Fixed in
README.mdandtests/e2e/run.js, in the suggested wording. - Fixed: the Welcome well…
Review: FAIL (needs-rework)
README.md(the Chrome-suite paragraph on sneak/AutistMask#668) and the comment opening that section oftests/e2e/run.jssay: "The…