-
Notifications
You must be signed in to change notification settings - Fork 278
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Need support for periods in an option string? #147
Comments
Got it. This is probably just a parsing point, I probably didn't intend support for periods in an option string, but I never documented that and there's probably not a reason I can't. This is where the options are initially parsed. https://github.com/dthree/vorpal/blob/master/lib/option.js#L15 I'll try to look further into it when I get a sec, but feel free to look around. |
@PeterDCarter he was referring to adding necessary boilerplate (or figuratively trash) to the code, obscuring its clarity. |
@dthree Why would you want to pollute your own code? |
Ha sorry, yeah. Did that in a rush |
let say I want options like
--vpn.user=bob
;vpn.host=sub.domain.net
; ...adding
would throw an error as
options.vpn
is not setso we need to do
even we don't have usage for vpn option.
this pollute the help generation, is prone to making errors and is kind of a boiler plate
maybe we could use something like https://www.npmjs.com/package/objectkit to auto create path ?
The text was updated successfully, but these errors were encountered: