This is an automated email from the ASF dual-hosted git repository.

paleolimbot pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/arrow-nanoarrow.git


The following commit(s) were added to refs/heads/main by this push:
     new 2d5276a5 chore(ci): Ensure clang-format version is consistent in 
pre-commit (#696)
2d5276a5 is described below

commit 2d5276a5feed4f4139e4993e88f6dc62a4f1cb27
Author: Dewey Dunnington <[email protected]>
AuthorDate: Thu Dec 26 21:51:55 2024 -0600

    chore(ci): Ensure clang-format version is consistent in pre-commit (#696)
    
    Before this PR, running `pre-commit run --all-files` was not sufficient
    to result in passing CI (particularly on MacOS, where the brew installed
    clang-format is considerably newer than the CI-installed clang-format).
    This PR just makes the version that had been enforced in CI part of the
    configuration.
---
 .pre-commit-config.yaml | 13 +++++--------
 1 file changed, 5 insertions(+), 8 deletions(-)

diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml
index 9ba75072..116bac10 100644
--- a/.pre-commit-config.yaml
+++ b/.pre-commit-config.yaml
@@ -29,13 +29,12 @@ repos:
       exclude: "^r/.*?/_snaps/.*?.md$"
     - id: trailing-whitespace
       exclude: "^r/.*?/_snaps/.*?.md$"
-  - repo: https://github.com/pocc/pre-commit-hooks
-    rev: v1.3.5
+  - repo: https://github.com/pre-commit/mirrors-clang-format
+    rev: v16.0.6
     hooks:
-      - id: clang-format
-        args: [-i]
-        types_or: [c, c++]
-        exclude: "(^thirdparty/.*$)|(flatcc_generated.h)"
+    - id: clang-format
+      types_or: [c, c++]
+      exclude: "(^thirdparty/.*$)|(flatcc_generated.h)"
   - repo: https://github.com/cheshirekow/cmake-format-precommit
     rev: v0.6.13
     hooks:
@@ -75,5 +74,3 @@ repos:
        additional_dependencies: ['meson==1.6.0']
        entry: meson format -i
        files: meson.build
-
-exclude: "^dist"

Reply via email to