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

Enable the wrap_help feature of clap #1746

Merged
merged 1 commit into from
Aug 29, 2024

Conversation

alexcrichton
Copy link
Member

This makes the --help and -h messages much more readable on terminals that don't happen to be hundreds of characters wide.

This is an example of wasm-tools addr2line --help before this PR

Screenshot 2024-08-29 at 9 42 16 AM

and after this PR on the same terminal

Screenshot 2024-08-29 at 9 42 48 AM

Longer explanations of flags are wrapped better and the description of the command is itself word-wrapped which is much easier to read.

This makes the `--help` and `-h` messages much more readable on
terminals that don't happen to be hundreds of characters wide.
alexcrichton added a commit to alexcrichton/wasmtime that referenced this pull request Aug 29, 2024
This is similar to bytecodealliance/wasm-tools#1746 which makes the help
text easier to read on various terminal widths.
@abrown abrown added this pull request to the merge queue Aug 29, 2024
Merged via the queue into bytecodealliance:main with commit 0d8caf3 Aug 29, 2024
29 checks passed
github-merge-queue bot pushed a commit to bytecodealliance/wasmtime that referenced this pull request Aug 29, 2024
* Enable clap's `wrap_help` feature by default

This is similar to bytecodealliance/wasm-tools#1746 which makes the help
text easier to read on various terminal widths.

* Fix `cargo deny`
github-merge-queue bot pushed a commit to bytecodealliance/wasmtime that referenced this pull request Aug 29, 2024
* Enable clap's `wrap_help` feature by default

This is similar to bytecodealliance/wasm-tools#1746 which makes the help
text easier to read on various terminal widths.

* Fix `cargo deny`
@alexcrichton alexcrichton deleted the wrap-help branch September 4, 2024 18:33
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