-
Notifications
You must be signed in to change notification settings - Fork 327
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Show ✗ when choosing ‘no’ in interactive mode #553
Comments
We should print an ✗ when the user chooses no. That's a bug (notice how it shows an ✗ if you kill the process, e.g. with Cmd + C, during a prompt). PR Welcome. As for implying |
Alright fair enough, then I'll concentrate only on the ✗ part! |
Hmmm this seems to be rather a problem with const prompts = require('prompts');
(async () => {
const response = await prompts({
type: 'confirm',
name: 'value',
message: 'Do you want to upgrade: stuff?',
initial: true
});
console.log(response.value);
})(); Gives me
while the last two are fine the first two are very weird 🤔 EDIT: opened terkelg/prompts#180 |
Agreed! Thanks for following up on this. |
v12.5.0
6.9.2
3.1.13
Just curious about people's opinions, when choosing ‘no’ in interactive mode, we still put the green check mark in the front - how about we change that to a red x in that case?
Before:
After:
Happy to create a PR if you like it.
Also: Should we imply
-u
when using interactive mode? I find it kind of counter intuitive when it asks meDo you want to upgrade: eslint ^5.16.0 → ^6.0.1? … yes
and since I didn’t run it with-u
it’s still on dry-run… 🤔The text was updated successfully, but these errors were encountered: