Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[FEATURE] Upgrade to clap v3 with declarative syntax #91

Closed
svenstaro opened this issue Feb 23, 2022 · 3 comments · Fixed by #109
Closed

[FEATURE] Upgrade to clap v3 with declarative syntax #91

svenstaro opened this issue Feb 23, 2022 · 3 comments · Fixed by #109
Assignees
@oknozor
Labels
enhancement New feature or request

Comments

@svenstaro
Copy link
Contributor

clap v3 supports nice declarative typesafe syntax. See here for an example: https://github.com/svenstaro/miniserve/blob/67e96523872eb5e94c6b828fc8c2a87646c1c5b2/src/args.rs#L12

Not only does this make the tool safer to use but it should reduce code and also allow for easy use for clap_mangen in order to easily solve #14.
@svenstaro svenstaro added the enhancement New feature or request label Feb 23, 2022
@oknozor
Copy link
Owner

oknozor commented Feb 23, 2022

I have almost finished this already. Did not push anything yet though.
I probably get this done by the week-end.

@svenstaro
Copy link
Contributor Author

Very cool!

@svenstaro
Copy link
Contributor Author

@oknozor any update on this? :)

@oknozor oknozor mentioned this issue Apr 5, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@oknozor oknozor

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Chore/update clap oknozor/toml-bombadil
2 participants
@svenstaro @oknozor