dthree / vorpal

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

Command names with capital letters #15

Closed AljoschaMeyer closed 8 years ago

AljoschaMeyer commented 9 years ago

Adding a command with capital letters does not work.

Running it (either correctly written or in lowercase) yields Invalid Command. Showing Help:. The help does include this:

fooBar                undefined

If forcing all-lowercase command names is intentional, a notification when an invalid command is added would be nice. If its unintentional: well, more stuff to fix =)

vorpal.command('fooBar')
  .action(function(args, cb) {
    this.log 'fooBar';
    cb();
});

* Not my actual code, please ignore js errors produced by my blissful, coffeescript-induced ignorance.

dthree commented 9 years ago

It was intentional, but not the most intelligent foresight. I should change this.

dthree commented 8 years ago

This is fixed now.