mqy commented on a change in pull request #8953:
URL: https://github.com/apache/arrow/pull/8953#discussion_r546073470
##########
File path: rust/README.md
##########
@@ -110,6 +110,9 @@ PR be sure to run the following and check for lint issues:
cargo +stable fmt --all -- --check
```
+We can use "git pre-commit hook" to automate this process: copy or soft link
[pre-commit.sh](pre-commit.sh)
+as file `.git/hooks/pre-commit`.
Review comment:
@jorgecarleitao good idea!
I have tried this way but finally aborted.
One of the reason is: one line command for copy/paste is not enough -- if
the pre-commit file already exists, copy/link may fail or override them
unexpectedly, thus additional check/prompt would make the commands lengthy.
Anyway, let me try again.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
