Add required options to the usage help text

Tyrrrz / CliFx

Class-first framework for building command-line interfaces

MIT License

1.5k stars 61 forks source link

Add required options to the usage help text #35

Closed dgarcia202 closed 4 years ago

dgarcia202 commented 4 years ago

implements behavior described in #10

The option value shown in the usage line can be set using a new option property SampleValue. As it is optional, when not specified, option's Name will be used instead.

I understand that maintainers could be against adding a new property to command option for that purpose, in that case, I can revert that part and use whatever value you may suggest.

Unit tests added.

Tyrrrz commented 4 years ago

Hey, thanks for your PR! I've looked over and left some small comments.

dgarcia202 commented 4 years ago

Thank you very much for taking the time to review my PR. Pushed some changes following your suggestions. I will be more than happy to look into any other comment you may have.

Tyrrrz commented 4 years ago

Thank you! Everything looks good, will merge now. 🎉