These are chat archives for kbknapp/clap-rs

14th
Aug 2015
Alexander Kuvaev
@Vinatorul
Aug 14 2015 19:56
Hello, I want to work on #169. So what if all settings will be passed via setting method and AppOptions enum. I mean author, version, etc, too.
Kevin K.
@kbknapp
Aug 14 2015 20:12
Awesome, thanks! I thought about doing that, and I think it's totally possible, but in my mind there is a bit of a difference between settings that affect how App operates and information that doesn't affect App but just fills in the blanks so to speak.
For this initial implementation I'd like to just stick with the book value methods, we can always add the others at a later point... But we can't remove anything without breaking semver
Also, I just finished #170 so I'll push and merge that so you can rebase. Then once both are done we'll cut the 1.2 release, :D
Alexander Kuvaev
@Vinatorul
Aug 14 2015 20:18
May be it should be setting and info methods. So I`ll add a setting method, than we can discuss it in PR
Kevin K.
@kbknapp
Aug 14 2015 20:19
OK cool thanks!
I'll merge #170 once I get home in a bit
Alexander Kuvaev
@Vinatorul
Aug 14 2015 21:21
Hm, what about docs, I need to move them to enum defenition?
to move examples*
Alexander Kuvaev
@Vinatorul
Aug 14 2015 22:34
I`ve crated PR #173, will return on sunday. Waiting for your merge.
Kevin K.
@kbknapp
Aug 14 2015 23:34
I made some notes on the PR, looks great though and once those changes are made it'll be ready to merge! :+1:
Alexander Kuvaev
@Vinatorul
Aug 14 2015 23:35
I fixed that