[y/N] prompt: unrecognized answers get no feedback, and Ctrl-D leaves the cancel message on the prompt line
I maintainer di solito rispondono entro 2 giorni
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 2/5
- Tempo stimato
- 1-3 ore
- Idoneità per principianti
- 78/100
Direzione di ricerca
Inizia leggendo l’helper di conferma condiviso in rocm_core::confirm, introdotto da #769/#782, e controlla i suoi chiamanti per rocm uninstall, rocm fix e la richiesta di sovrascrittura dell’SDK. Riproduci i casi di risposta non riconosciuta e Ctrl-D in uno pseudo-terminale, poi verifica che l’helper fornisca il feedback richiesto e termini la riga del prompt in caso di EOF.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Current behavior
The shared [y/N] prompt treats anything other than y/yes as No, including EOF, and gives no feedback in two cases. Observed with rocm uninstall on a pseudo-terminal:
Proceed with uninstall? [y/N]: maybe
uninstall cancelled
An unrecognized answer is declined exactly like n, with no hint that the answer was not understood. On Ctrl-D (EOF) the cancel message is printed on the prompt line, because no newline was echoed:
Proceed with uninstall? [y/N]: uninstall cancelled
The parsing (y/yes, case-insensitive; everything else is No) is unchanged by the helper that #769/#782 introduce in rocm_core::confirm, so every caller inherits it.
Expected behavior
For an unrecognized answer, either re-prompt or say that it is being treated as No. On EOF, end the prompt line before printing the cancel message.
Additional context
Changing the shared helper changes output for rocm fix and the SDK-overwrite prompt as well as uninstall, which is why it was left out of the behavior-preserving refactor.
Your environment
- rocm-cli version:
rocm-cli 0.1.0, built from the #782 branch (910b1d8). That branch only refactors the[y/N]prompt helper (#769) without changing its logic; not re-run onmain. - Platform: WSL2 (Linux 6.18)
- Lingua principale
- Rust
- Stelle
- 49
- Fork
- 14
- Merge medio
- 3g 6h
- PR unite (30g)
- 183
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Ha un modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di ROCm/rocm-cli
-
bug installation
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
I maintainer di solito rispondono entro 2 giorni
-
bug documentation
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
I maintainer di solito rispondono entro 2 giorni
-
documentation
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
I maintainer di solito rispondono entro 2 giorni
-
enhancement ux-opportunities
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
I maintainer di solito rispondono entro 2 giorni
-
enhancement ux-opportunities
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
I maintainer di solito rispondono entro 2 giorni
Tutte le issue di ROCm/rocm-cli
Issue simili
-
spurctld: a user with no usage gets a lower fair-share factor than a user with a little usageAperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
I maintainer di solito rispondono entro 1 giorno
-
area:ci kind:bug priority:p3 ready
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
I maintainer di solito rispondono entro 1 giorno
-
tech-debt
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
srikanth235/centraid#1155 ·
I maintainer di solito rispondono entro 1 giorno
-
dotfiles: failed batch untrack removes earlier tracking declarationsForse già presa @jdx l’ha presa oggi. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
I maintainer di solito rispondono entro 2 giorni