This is an automated email from the ASF dual-hosted git repository.
kou pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/arrow-java.git
The following commit(s) were added to refs/heads/main by this push:
new 89217c8e GH-520: [CI] Add cmake-format to pre-commit (#521)
89217c8e is described below
commit 89217c8ed7222e4f30e0190c65cf64ee06d9ec06
Author: Sutou Kouhei <[email protected]>
AuthorDate: Wed Jan 15 12:51:05 2025 +0900
GH-520: [CI] Add cmake-format to pre-commit (#521)
Fixes GH-520.
The configuration file is borrowed from apache/arrow.
---
.cmake-format.py | 76 +++++++++++++++++++++++++++++++++++++++++++++++++
.pre-commit-config.yaml | 4 +++
2 files changed, 80 insertions(+)
diff --git a/.cmake-format.py b/.cmake-format.py
new file mode 100644
index 00000000..b8fc8939
--- /dev/null
+++ b/.cmake-format.py
@@ -0,0 +1,76 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor license agreements. See the NOTICE file
+# distributed with this work for additional information
+# regarding copyright ownership. The ASF licenses this file
+# to you under the Apache License, Version 2.0 (the
+# "License"); you may not use this file except in compliance
+# with the License. You may obtain a copy of the License at
+#
+# http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing,
+# software distributed under the License is distributed on an
+# "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+# KIND, either express or implied. See the License for the
+# specific language governing permissions and limitations
+# under the License.
+
+# cmake-format configuration file
+# Use `archery lint --cmake-format --fix` to reformat all cmake files in the
+# source tree
+
+# -----------------------------
+# Options affecting formatting.
+# -----------------------------
+with section("format"):
+ # How wide to allow formatted cmake files
+ line_width = 90
+
+ # How many spaces to tab for indent
+ tab_size = 2
+
+ # If a positional argument group contains more than this many arguments,
+ # then force it to a vertical layout.
+ max_pargs_hwrap = 4
+
+ # If the statement spelling length (including space and parenthesis) is
+ # smaller than this amount, then force reject nested layouts.
+ # This value only comes into play when considering whether or not to nest
+ # arguments below their parent. If the number of characters in the parent
+ # is less than this value, we will not nest.
+ min_prefix_chars = 32
+
+ # If true, separate flow control names from their parentheses with a space
+ separate_ctrl_name_with_space = False
+
+ # If true, separate function names from parentheses with a space
+ separate_fn_name_with_space = False
+
+ # If a statement is wrapped to more than one line, than dangle the closing
+ # parenthesis on it's own line
+ dangle_parens = False
+
+ # What style line endings to use in the output.
+ line_ending = 'unix'
+
+ # Format command names consistently as 'lower' or 'upper' case
+ command_case = 'lower'
+
+ # Format keywords consistently as 'lower' or 'upper' case
+ keyword_case = 'unchanged'
+
+# ------------------------------------------------
+# Options affecting comment reflow and formatting.
+# ------------------------------------------------
+with section("markup"):
+ # enable comment markup parsing and reflow
+ enable_markup = False
+
+ # If comment markup is enabled, don't reflow the first comment block in
+ # eachlistfile. Use this to preserve formatting of your
+ # copyright/licensestatements.
+ first_comment_is_literal = True
+
+ # If comment markup is enabled, don't reflow any comment block which
+ # matches this (regex) pattern. Default is `None` (disabled).
+ literal_comment_pattern = None
diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml
index dd365b15..0a762414 100644
--- a/.pre-commit-config.yaml
+++ b/.pre-commit-config.yaml
@@ -16,6 +16,10 @@
# under the License.
repos:
+ - repo: https://github.com/cheshirekow/cmake-format-precommit
+ rev: v0.6.13
+ hooks:
+ - id: cmake-format
- repo: https://github.com/pre-commit/pre-commit-hooks
rev: cef0300fd0fc4d2a87a85fa2093c6b283ea36f4b # v5.0.0
hooks:
