Adds a GH Actions workflow to run clippy, and attach any clippy warnings to PRs/commits as GitHub annotations
Builds take around ~30min with no cache, and around ~5min with a warm cache, on the default Actions runners.
Using cargo2nix here could potentially work, but (at least from my reading of their docs) it requires another lockfile (Cargo.nix) that could fall out of sync with Cargo.lock---and would make contributing to this repo more complicated. It's something we can look into down the line, but for now I don't think the extra complexity is really necessary.
To do:
[ ] Check that nix flake check runs the Cargo tests locally
[ ] Configure nix flake check to run Clippy locally
This PR:
wasm_bindgen
, until they're fixed upstream (in https://github.com/rustwasm/wasm-bindgen/issues/2774)Builds take around ~30min with no cache, and around ~5min with a warm cache, on the default Actions runners.
Using
cargo2nix
here could potentially work, but (at least from my reading of their docs) it requires another lockfile (Cargo.nix
) that could fall out of sync withCargo.lock
---and would make contributing to this repo more complicated. It's something we can look into down the line, but for now I don't think the extra complexity is really necessary.To do:
nix flake check
runs the Cargo tests locallynix flake check
to run Clippy locally