The comment on commandWait in internal/cli/integration_test.go said script/test gives the package 30 seconds. script/test sets no time limit; the test phase of the Dockerfile sets one with go test -timeout. The comment now names that place and the flag instead of a number. Only that comment changes, plus the TODO.md entry.
Judgement call: of the two options the issue allows, I took the one without a number.
Model: opus-5-5
Fixes https://git.eeqj.de/sneak/secret/issues/195.
The comment on `commandWait` in `internal/cli/integration_test.go` said `script/test` gives the package 30 seconds. `script/test` sets no time limit; the test phase of the `Dockerfile` sets one with `go test -timeout`. The comment now names that place and the flag instead of a number. Only that comment changes, plus the `TODO.md` entry.
Judgement call: of the two options the issue allows, I took the one without a number.
Model: opus-5-5
The comment said script/test gives the package 30 seconds. script/test
sets no time limit; the test phase of the Dockerfile sets one with
go test -timeout. The comment now names that place instead of a number.
Model: opus-5-5
clawbot
self-assigned this 2026-10-15 20:03:45 +02:00
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.
Fixes #195.
The comment on
commandWaitininternal/cli/integration_test.gosaidscript/testgives the package 30 seconds.script/testsets no time limit; the test phase of theDockerfilesets one withgo test -timeout. The comment now names that place and the flag instead of a number. Only that comment changes, plus theTODO.mdentry.Judgement call: of the two options the issue allows, I took the one without a number.
Model: opus-5-5
Review passed: the new
commandWaitcomment is true of the currentnextand theTODO.mdentry records the fix accurately.Model: opus-5-5