pre-commit-rust
pre-commit-rust copied to clipboard
doublify
Rust hooks for pre-commit
Hi, I've added cargo-test to the hooks list, I needed it myself so I've thought I'd contribute.
Makes it easier for the user to know how to set up all the hooks available.
I noticed that the current hooks use `language: system`, but it would be nice if the hooks used `language: rust` instead (e.g. this is needed for pre-commit.ci)
I'm tinkering with a project, right now, which compiles fine with stable rust, _but_ its rustfmt configuration uses a whole bunch of options that are only available with nightly rustfmt....
should this hook provide a way to disable the 'deny warnings' flag? many codebases (including my own) use compiler flags to set the warning level for clippy lints. the 'deny...
This is a new one to me, but seems it'd be a very useful pre-commit hook, i.e. 'you just removed the last use of this dependency': https://github.com/est31/cargo-udeps
The double dash needs to come before rustfmt args. Also, specifying it when Cargo.toml already specifies the edition will result in a warning.
https://github.com/rust-lang/rust-clippy/issues/1495 and https://github.com/rust-lang/rust-clippy/issues/4612 for details. In short, clippy is not guaranteed to output results unless a few conditions are satisfied. The easiest of which is running `cargo clean` before clippy....
I have repository that has both Python and Rust code inside. In the root of the repository I have `.pre-commit-config.yaml` with: ```yaml - repo: https://github.com/doublify/pre-commit-rust rev: master hooks: - id:...
