Skip to content

fix(ValueEnum): Detailed help for ValueEnum variant #6096 #6101

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

Merged
merged 2 commits into from
Aug 12, 2025

Conversation

smoqadam
Copy link
Contributor

@epage
Copy link
Member

epage commented Aug 11, 2025

Note that the intention of #6096 is not to add separate short vs long help (we only show any help in long help) but to change the derive from reporting the doc comment summary to the full doc comment.

@smoqadam
Copy link
Contributor Author

Note that the intention of #6096 is not to add separate short vs long help (we only show any help in long help) but to change the derive from reporting the doc comment summary to the full doc comment.

To clarify my understanding based on your last comment: help should render the full doc comment of a PossibleValue, right? I’m not sure how close (or far) my recent changes are from this goal.

@smoqadam smoqadam marked this pull request as ready for review August 12, 2025 07:00
@epage
Copy link
Member

epage commented Aug 12, 2025

Could you clean up the commits for how this should be reviewed and merged?

@smoqadam smoqadam force-pushed the fix/value-enum-detailed-help branch from cc012d2 to 9fdc172 Compare August 12, 2025 16:46
@smoqadam smoqadam force-pushed the fix/value-enum-detailed-help branch from 9fdc172 to 3225e5e Compare August 12, 2025 17:27
@epage epage force-pushed the fix/value-enum-detailed-help branch 2 times, most recently from f28876d to a8b8289 Compare August 12, 2025 17:49
@epage epage merged commit 9fd06f2 into clap-rs:master Aug 12, 2025
25 checks passed
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.

2 participants