(#80) - Use nomnom for argument parsing#80
Conversation
There was a problem hiding this comment.
Splitting the usage strings would become very messy, even more since adding newlines would stop 'wordwrap' from working. I decided to just turn length checking off instead.
There was a problem hiding this comment.
For the usage strings, you can just precede your super-long strings with:
/* jshint maxlen:false */and then afterwards:
/* jshint maxlen:80 */There was a problem hiding this comment.
Ah, yes, that's the better solution.
There was a problem hiding this comment.
Seems like jshint/jshint#1313 prevents me from doing that (my best guess is that it sees the node option as being inside an anonymous function). Wrapping the argument parsing inside a function and disabling maxlen for that function only works, so I used that as a workaround.
|
+1 except for that nit about |
|
yargs is the other one I looked into, it's the more direct successor to optimist but I actually liked nomnom's api better. |
|
Fixed all the things commented upon, and merged: 4de5ced Thanks for the elaborate reviews! |
|
Yeah man, no prob. Everything you've been doing here has been really cool. :) |
Why the switch