Allow setting of non-boolean options #3

Merged
merged 5 commits into from Dec 14, 2011

Conversation

Projects
None yet
3 participants

skenqbx commented Dec 9, 2011

Hello David,

a small addition to allow setting of non-boolean options, e.g. node app.js --set maxlen:80,regexp:false,node.
Also added cumulated jslint options output to command-line.

I left --include & --exclude untouched, although they are not really needed anymore.

Thanks for the quick response on my last pull request.

Regards,
Malte

tomterl commented Dec 9, 2011

I tested the code in this request on two machines (node 0.6.2 and 0.6.4); with maltes latest commit (92c6f0c) everything works as expected.

davidmiller added a commit that referenced this pull request Dec 14, 2011

Merge pull request #3 from Joocom/master
Allow setting of non-boolean options

@davidmiller davidmiller merged commit 876422c into davidmiller:master Dec 14, 2011

Owner

davidmiller commented Dec 14, 2011

That's pretty useful, thanks :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment