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

List subcommand accepts redundant arguments #3632

Closed
celinval opened this issue Oct 22, 2024 · 0 comments · Fixed by #3729
Closed

List subcommand accepts redundant arguments #3632

celinval opened this issue Oct 22, 2024 · 0 comments · Fixed by #3729
Assignees
Labels
[C] Bug This is a bug. Something isn't working. [E] User Experience An UX enhancement for an existing feature. Including deprecation of an existing one.

Comments

@celinval
Copy link
Contributor

I tried using the following command line invocation:

kani list -h

with Kani version: 0.56.0

I expected to see this happen: Just a few options to show up

Instead, this happened: Options such as --unwind, --cbmc-args, --concrete-playback, and so on show up as possible arguments.

@celinval celinval added [C] Bug This is a bug. Something isn't working. [E] User Experience An UX enhancement for an existing feature. Including deprecation of an existing one. labels Oct 22, 2024
@carolynzech carolynzech self-assigned this Oct 22, 2024
github-merge-queue bot pushed a commit that referenced this issue Nov 20, 2024
Improvements to the list subcommand.

- Print the location of the JSON file for `--format=json` (#3633)
- Use `CommonArgs` instead of `VerificationArgs` (#3632)
- Add a Markdown option. The pretty table is nice for smaller packages,
but as we progress with the standard library verification effort, it's
too massive to be easily readable. In this case, it's better to print a
markdown table and then just read it in a markdown editor. (Once this PR
is merged, the [kani list workflow
step](model-checking/verify-rust-std#146) will
use the markdown option).

Resolves #3632, #3633

By submitting this pull request, I confirm that my contribution is made
under the terms of the Apache 2.0 and MIT licenses.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[C] Bug This is a bug. Something isn't working. [E] User Experience An UX enhancement for an existing feature. Including deprecation of an existing one.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants