search

siokas / denomander

Deno command-line interfaces made easy
https://doc.deno.land/https/deno.land/x/denomander/docs.ts
MIT License
149 stars 8 forks source link

Required options scope are program wise instead of command wise #17

Closed Willovent closed 4 years ago

Willovent commented 4 years ago

When a required options is set for a command, it must be provide for all commands instead of just the one adding it 

deno run https://raw.githubusercontent.com/siokas/denomander/master/example.ts clone folder

outputs : Error: Required option is not specified!

deno run https://raw.githubusercontent.com/siokas/denomander/master/example.ts clone folder -p 8080

outputs: The repo is cloned into: folder

siokas commented 4 years ago

In next release, I plan to make every option to belong on a specific command. So every option (including required options) are going to be command wise.

I inspired from laravel's artisan commands to to so. Do you think that this is not needed and options should be program wise?

Willovent commented 4 years ago

I think this is needed and it's great to see it's already on your todo !