-
remove cargo fmt’ from CI and instead [add a#[test]](https://github.com/matklad/tracing-span-tree/blob/e8864ad0274bc9342b88e1ee3388f7b6a36a47f2/xtask/tests/tidy.rs#L4). It’s better to encode all “correctness” criteria intocargo test`, as that can easily be run locally and is independent form CI provider.
-
CodeRabbit
CodeRabbit: AI Code Reviews for Developers. Revolutionize your code reviews with AI. CodeRabbit offers PR summaries, code walkthroughs, 1-click suggestions, and AST-based analysis. Boost productivity and code quality across all major languages with each PR.
-
audit for security audits - Separate from regular CI since it only matters for specific changes or when new critical issues come out.
-
rust-next to verify nothing breaks with new Rust versions - Separate from regular CI since the probability of a PR breaking on a new Rust version is pretty low (its pretty low this will ever fail, but just in case)
-
For more details on the idea, see the discussion on clap