node-reinstall icon indicating copy to clipboard operation
node-reinstall copied to clipboard
verified publisher icon brock

Add --only-remove flag

Open hanoii opened this issue 8 years ago • 1 comments

hanoii avatar May 12 '17 13:05 hanoii

Thanks so much for creating a pull request! You are an awesome person, and you should feel good about yourself!! 😄 On to the PR...

I'm okay with the concept of adding an option for --only-remove, but when I tested this, the prompts felt just like the node-reinstall prompts, and I could imagine a new user feeling confused. I think this task deserves it's own prompts so that the user knows exactly what they are getting.

Here's an example. For a new user trying to uninstall node, this might be confusing: screen shot 2017-05-17 at 7 37 55 pm

This made it look like we were going to re-install Node.js, even though we'd explicitly said --only-remove.

If we go this route, I think we should move the remove functionality into a separate function and make it a little more clear that we aren't going to re-install node or npm. That way we could do more customized messaging to the user so they know this is just a wipe without a re-install.

brock avatar May 18 '17 00:05 brock