search

rust-embedded / riscv

Low level access to RISC-V processors
822 stars 160 forks source link

Added clippy to the CI workflow #123

Closed romancardenas closed 1 year ago

romancardenas commented 1 year ago

I've been working with this crate for a while and noticed that it didn't include clippy as part of the CI process. This PR checks that clippy does not trigger any warning for new contributions. I also ran cargo clippy --fix to "fix" the crate.

If you think this is useful, I can adapt other repos (e.g., riscv-rt or e310x). Cheers!

almindor commented 1 year ago

I was also wondering about clippy. Thanks! Will review when I get a moment

almindor commented 1 year ago

bors r+

bors[bot] commented 1 year ago

Build succeeded: