kou commented on pull request #9045:
URL: https://github.com/apache/arrow/pull/9045#issuecomment-804431484


   @MarcoGorelli I found some problems:
   
     * `python run-cmake-format.py` (no `paths` arguments) doesn't work because 
`PATTERNS` data aren't suitable for `glob()`.
     * We also need exclude `cpp/src/arrow/util/config.h.cmake` because it's 
not a CMake file. (It may be better that we rename the `.cmake` extension of 
the file.)
   
   I've fixed them and rebased on the current master.
   
   Could you check this? If it's OK to you, I'll merge this.


-- 
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:
us...@infra.apache.org


Reply via email to