alamb commented on code in PR #2444:
URL: https://github.com/apache/arrow-datafusion/pull/2444#discussion_r865936702


##########
dev/format-code.sh:
##########
@@ -0,0 +1,22 @@
+#!/bin/bash

Review Comment:
   I would be quite happy to help those who wanted to install a pre-commit hook 
but am opposed to anything that required it / made it the default in this 
repository as various developers have their own ideas of ideal workflows
   
   I suggest adding the three scripts as suggested
   ```
   ci/scripts/rust_fmt.sh
   ci/scripts/rust_clippy.sh
   ci/scripts/toml_fmt.sh
   ```
   that are invoked from the CI jobs (and thus remain in sync)
   
   And then people can mix/match however they want to run them 



-- 
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]

Reply via email to