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

Enable the rust_2018_idioms lint as warning #171

Merged
merged 1 commit into from
Jun 13, 2022
Merged

Enable the rust_2018_idioms lint as warning #171

merged 1 commit into from
Jun 13, 2022

Commits on Jun 13, 2022

  1. Enable the rust_2018_idioms lint as warning 

    This should become the default in the future:
rust-lang/rust#54910

Reason this is put in lib.rs instead of CI: to ensure local invocations
of cargo build and cargo clippy pick up the warning.

Reason this is made a warning: so that code builds and runs anyway. On
the CI, the `-D warnings` clippy argument will treat it as a hard error.
    @hhirtz
    hhirtz committed Jun 13, 2022
    Configuration menu
    Copy the full SHA
    bd9f9b1 View commit details
    Browse the repository at this point in the history