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.

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

Upgrade to Clap 4. #78

Merged
merged 1 commit into from Mar 12, 2023
Merged

Upgrade to Clap 4. #78

merged 1 commit into from Mar 12, 2023

Conversation

arusahni
Copy link
Owner

This jumps two major versions. Notably, this switches from the builder pattern to the Derive API. In the past few years, Clap has actively distanced themselves from the Yaml-based approach we were taking advantage of for manpage generation. Thankfully, there's a clap_mangen library that generates the manpage from the command object.

The manpage does advise the user to use git-req instead of git req. This is an issue(?) with clap_mangen, which would be fixed by clap-rs/clap#4758.

This jumps two major versions. Notably, this switches from the builder
pattern to the Derive API. In the past few years, Clap has actively
distanced themselves from the Yaml-based approach we were taking
advantage of for manpage generation. Thankfully, there's a `clap_mangen`
library that generates the manpage from the command object.
@arusahni arusahni merged commit d147478 into master Mar 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant