search

dthree / vorpal

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

Fixes multiple matching commands for autocomplete #142

Closed jeff-1amstudios closed 8 years ago

jeff-1amstudios commented 8 years ago

hey @dthree this is a small set of the autocomplete changes I made against a previous build a few months ago - I still see some of these problems in 1.10.10, this fixes those cases and adds tests to make sure we never regress

dthree commented 8 years ago

Thanks! :+1:

jeff-1amstudios commented 8 years ago

Would you be able to publish a new build to npm which includes this fix? In the meantime I've been referencing this git revision in my package.json and have had reports of failing installs because users didn't have git installed.

thanks! Jeff

On Thu, Apr 7, 2016 at 10:11 AM, dc notifications@github.com wrote:

Thanks! [image: :+1:]

— You are receiving this because you authored the thread. Reply to this email directly or view it on GitHub https://github.com/dthree/vorpal/pull/142#issuecomment-206998101

jeff-1amstudios commented 8 years ago

Ping :) Could you publish the latest build to npm with this fix?

dthree commented 8 years ago

Hey Jeff. Doing it now :)

dthree commented 8 years ago

Ok, this is published in v1.11.1.

jeff-1amstudios commented 8 years ago

great, thanks!