Copilot commented on code in PR #868:
URL: https://github.com/apache/incubator-graphar/pull/868#discussion_r2844103211
##########
.pre-commit-config.yaml:
##########
@@ -87,4 +87,21 @@ repos:
language: system
files: ^maven-projects/spark/
pass_filenames: false
- require_serial: true
\ No newline at end of file
+ require_serial: true
+
+ - id: cargo-fmt
+ name: Rust cargo fmt
+ entry: bash -c 'cd rust && cargo fmt --all'
+ language: system
+ files: ^rust/.*\.(rs|toml|lock)$
Review Comment:
The `cargo-fmt` hook is configured to run on `.toml` and `.lock` changes,
but `cargo fmt` only formats Rust source files. Consider narrowing the `files`
regex to `^rust/.*\.rs$` (or switching to `types: [rust]`) to avoid unnecessary
hook runs on non-Rust files.
```suggestion
files: ^rust/.*\.rs$
```
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
