Running cargo check and cargo clippy can be considered redundant as the latter one includes everything the former one does.
Running cargo fmt -- --check is usually so fast that the CI runtime is dominated by the setup time, hence it seems advisable to just fold that into the Check CI job. (The ordering could be changed though. Rustfmt first weeds out failing builds AFAP but does not indicate whether there more than just formatting issues.)
Also, actions-rs/toolchain does support installing additional components so that no manual invocations of rustup are not necessary.
Running
cargo checkandcargo clippycan be considered redundant as the latter one includes everything the former one does.Running
cargo fmt -- --checkis usually so fast that the CI runtime is dominated by the setup time, hence it seems advisable to just fold that into the Check CI job. (The ordering could be changed though. Rustfmt first weeds out failing builds AFAP but does not indicate whether there more than just formatting issues.)Also, actions-rs/toolchain does support installing additional components so that no manual invocations of rustup are not necessary.