Closed shuiyisong closed 3 months ago
This update simplifies the .pre-commit-config.yaml
by modifying the arguments for the clippy
and cargo-check
hooks. Now, both hooks utilize the --all-features
flag, ensuring that all features within the project are included in the checks without specifying additional flags.
Files | Change Summary |
---|---|
.pre-commit-config.yaml |
Updated clippy and cargo-check hooks to include --all-features . |
🐇✨ In the field where code does lie, Pre-commit hooks now reach the sky. With
--all-features
, they’ll check it all, Ensuring quality, standing tall. For every build and every run, Our code is safe; our coding’s fun! 🌸🚀
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 84.68%. Comparing base (
69bb7de
) to head (961bf73
).
I hereby agree to the terms of the GreptimeDB CLA.
Refer to a related PR or issue link (optional)
What's changed and what's your intention?
Align pre-commit config with our
Makefile
so that it won't check twice when commit and pushChecklist
Summary by CodeRabbit
--all-features
forclippy
andcargo-check
to ensure comprehensive code checks.