raulcd commented on code in PR #44529:
URL: https://github.com/apache/arrow/pull/44529#discussion_r1815134319
##########
.pre-commit-config.yaml:
##########
@@ -103,20 +104,23 @@ repos:
hooks:
- id: clang-format
name: C/GLib Format
+ alias: c-glib-format
files: >-
^c_glib/
- repo: https://github.com/pre-commit/mirrors-clang-format
rev: v14.0.6
hooks:
- id: clang-format
name: MATLAB (C++) Format
+ alias: matlab-format
files: >-
^matlab/src/cpp/
- repo: https://github.com/pre-commit/mirrors-clang-format
rev: v14.0.6
hooks:
- id: clang-format
name: Python (C++) Format
+ alias: python-format
Review Comment:
This should probably be better named `python-cpp-format` otherwise it seems
like it is formatting the Python code, which is done at the flake8 stage.
Probably the same for other languages like R or Matlab as this is formatting
the C++ code for those.
--
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]
