search

lit-robotics / libcamera-rs

Experimental Rust bindings for libcamera
Apache License 2.0
46 stars 15 forks source link

Fix ci warnings #7

Closed luigi311 closed 1 year ago

luigi311 commented 1 year ago

This fixes the CI warnings by updating some the actions and replacing the action-rs actions due to them all being abandoned.

I used the dtolnay/rust-toolchain for installing rust since that seems to be the most popular option that was recommended in the action-rs repos.

For the normal cargo commands it doesnt seem like there is any alternative but luckily this isnt doing much so there was no need for a replacement for that so this is just calling cargo directly.

I also updated the on conditions so it will run the build and test on all branches and PR except for .gitignore/readme changes so its possible to easily debug if a commit is broken without submitting a PR to main.

chemicstry commented 1 year ago

While at it, could you also add clippy step to fix #8?

luigi311 commented 1 year ago

Ok i added in clippy and fmt check

chemicstry commented 1 year ago

Thanks!