Anki-Android-Backend icon indicating copy to clipboard operation
Anki-Android-Backend copied to clipboard

Consider auto-running clippy on commit

Open david-allison opened this issue 5 months ago • 1 comments

For discussion

Note: this did produce unexpected formatting, so fmt was necessary

cargo clippy --fix --allow-dirty --allow-staged
cargo fmt

david-allison avatar Jun 14 '25 19:06 david-allison

works for me, it's possibly heavy but...if it's not in pre-commit it'll likely not be run

should be check-rust.sh or .bat though I think, so all our learnings are encapsulated (like, you have to run fmt, and you need to set the environment variable to make warnings errors etc)

also, there was all that work in Anki-Android repo about preserving staged files on failure, apparently that can be a problem

mikehardy avatar Jun 14 '25 19:06 mikehardy