-
Notifications
You must be signed in to change notification settings - Fork 14
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
feature: Print long help text for options #23
Conversation
85ad6a4
to
7ad8aa3
Compare
Fixes ConnorGray#20 I wrote my own `indent` function to avoid introducing any new dependencies (`indent` or `textwrap` crate).
7ad8aa3
to
4ab754c
Compare
This is great, thank you @ilyagr for these nice QoL contributions! 🧹 🙂 |
Thank you very much for merging this, #22, as well as starting and maintaining this project! I'm very glad to see that it is not abandoned. I use |
Here's the diff: https://github.com/martinvonz/jj/pull/3891/files#diff-241a36d22079376383ba2fbdd9921ba5b1492f2ce080a8b965373b52cf92656e. I'm just sharing it because I'm quite happy about it. It might take me a bit to actually merge that PR. |
That's awesome @ilyagr, I'm glad to see the new version is improving things for you! 🙂 Please let me know if there are any other features that would be useful to you! As a sneak peak for you as well: I'm currently working on a small overhaul to clap-markdown to use a Markdown AST representation internally instead of a raw string builder, which should make it much easier to add more user-customizable behavior to clap-markdown in a structured way. |
Just came here to work out how to fix this, and it's already fixed 🎉 . Thanks folks for making this library better. |
This fixes #20.
I separated this from #22 since there is a bit more logic here. The first commit is shared with #22, since test changes are confusing without that commit. I will rebase one of the PRs if you merge the other (unless you do it).
This duplicates janstarke#2, but is rebased onto this repo's main branch.