search

dthree / vorpal

Node's framework for interactive CLIs
http://vorpal.js.org
MIT License
5.64k stars 280 forks source link

Fix for undefined check #168

Closed alansouzati closed 8 years ago

alansouzati commented 8 years ago

Fixes #167

zakhenry commented 8 years ago

Oops that was my mistake, only possible concern is that for a prompt that has a default of false, the default won't be output, but that is an edge case that should be addressed after this fix is merged.

alansouzati commented 8 years ago

thanks for the heads-up on the false corner case @zakhenry. I've updated this PR with the fix for that as well.

alansouzati commented 8 years ago

hey @dthree would appreciate your insights here 😄

dthree commented 8 years ago

Looks really good, thanks for the PR! :+1:

dthree commented 8 years ago

Pushed in v1.11.14.