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.

Sign up for GitHub

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

Jump to bottom

Fix issue where cargo fmt --version would not display version info #5396

Merged
merged 1 commit into from
Jun 22, 2022
Merged

Fix issue where cargo fmt --version would not display version info #5396

merged 1 commit into from
Jun 22, 2022

Commits on Jun 21, 2022

  1. Fix issue where cargo fmt --version would not display version info 

    Fixes 5395

In PR 5239 we switched from using `structopt` to `clap`. It seems that
the default behavior for `clap` is to override the `--version` flag,
which prevented our custom version display code from running.

The fix as outlined in clap-rs/clap#3405 was
to set `#[clap(global_setting(AppSettings::NoAutoVersion))]` to prevent
clap from setting its own default behavior for the `--version` flag.
    @ytmimi
    ytmimi committed Jun 21, 2022
    Configuration menu
    Copy the full SHA
    17fea9a View commit details
    Browse the repository at this point in the history