-
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
Possible values listed for flags (and shouldn't be); cargo test
fails for the same reason
#19
Comments
ilyagr
changed the title
Possible values listed for flags,
Possible values listed for flags (and shouldn't be); Feb 1, 2024
cargo test
failscargo test
fails for the same reason
3 tasks
ilyagr
added a commit
to ilyagr/clap-markdown
that referenced
this issue
Jun 13, 2024
The user cannot write `--list false` for a flag. Fixes ConnorGray#19.
ilyagr
added a commit
to ilyagr/clap-markdown
that referenced
this issue
Jun 13, 2024
The user cannot write `--list false` for a flag. Fixes ConnorGray#19.
ilyagr
added a commit
to ilyagr/clap-markdown
that referenced
this issue
Jun 13, 2024
The user cannot write `--list false` for a flag. Fixes ConnorGray#19.
ilyagr
added a commit
to ilyagr/clap-markdown
that referenced
this issue
Jun 14, 2024
The user cannot write `--list false` for a flag. Fixes ConnorGray#19.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
When there is a flag defined with something like:
clap-markdown
currently adds "Possible values: true, false" to its help. This is undersired, since the user cannot do--list false
.This example is taken from the tests for this library. Currently, this results in the test failure because the (buggy) output is not shown in https://github.com/ConnorGray/clap-markdown/blob/16c42cdd5f65b815bc1672e0b0e2becf08a5f8f1/docs/examples/complex-app.md.
The correct output would be the one actually shown in the above link:
Instead,
clap-markdown
currently prints:I'm using clap 4.4.18. In case the version matters, here's my full
Cargo.lock
thatcargo test
generated for this repo:The text was updated successfully, but these errors were encountered: