ctubbsii commented on a change in pull request #263:
URL: https://github.com/apache/fluo-uno/pull/263#discussion_r542455929
##########
File path: bin/impl/commands.sh
##########
@@ -300,7 +299,7 @@ function uno_fetch_main() {
function uno_wipe_main() {
local yn
uno_kill_main
- read -r -p "Are you sure you want to wipe '$INSTALL'? " yn
+ read -r -p "Are you sure you want to wipe '$INSTALL'? [Y/n] " yn
Review comment:
I think it's less verbose to simply omit it, as it originally was. Both
`y` and `yes` work, and neither are case-sensitive. So, even if the user isn't
clear what options are available, the script is tolerant.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
