starred/cli
1
0
Fork 0
mirror of https://github.com/docker/cli.git synced 2026-08-28 02:24:15 -05:00
Code Issues Packages Projects Releases Wiki Activity
13,005 Commits 13 Branches 320 Tags
master
Commit Graph
4 Commits
Author SHA1 Message Date
Sebastiaan van Stijn fd3497683d internal/prompt: Confirm: don't wrap stdIn 
This was a remnant from when `PromptForConfirmation` accepted a `InStream`;
it was added in [moby@30b8f08], which got left behind when [cli@37ccc00]
updated its signature to accept a plain `io.Reader`.

While updating, also update the comment to provide more context on the reason
we're unconditionally using `os.Stdin` on Windows.

[moby@30b8f08]: https://github.com/moby/moby/commit/30b8f084436a2a1d5e8523fcd2c5ea64cc805224
[cli@37ccc00]: https://github.com/docker/cli/commit/37ccc00d0e14461bcf29e98669773625dfed2fce

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
 2026-07-07 14:19:44 +02:00
Sebastiaan van Stijn b74b7b3c40 internal/prompt: TestConfirm: don't use un-keyed structs 
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
 2025-04-11 12:23:24 +02:00
Sebastiaan van Stijn ecde8c38a5 internal/prompt: skip fmt.Printf and use writer directly 
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
 2025-04-11 12:23:21 +02:00
Sebastiaan van Stijn b37d84fd10 cli/command: move prompt utilities to separate package 
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
 2025-04-11 12:23:16 +02:00