apache / arrow

Apache Arrow is a multi-language toolbox for accelerated data interchange and in-memory processing
https://arrow.apache.org/
Apache License 2.0
13.7k stars 3.35k forks source link

GH-41688: [Dev] Include all relevant CMakeLists.txt files in cmake-format precommit hook #41689

Closed jorisvandenbossche closed 2 weeks ago

jorisvandenbossche commented 2 weeks ago

Rationale for this change

Some CMakeLists.txt files are not included in the pre-commit hook (causing failures on CI through archery if you rely on the pre-commit hook locally)

What changes are included in this PR?

Include all CMakeLists.txt files by default anywhere in the repo, and explicitly exclude the ones we don't want (vendored files).

In practice, compared to the current set of files covered by the hook, those new files are included in the search:

'cpp/CMakeLists.txt', 'java/CMakeLists.txt', 'matlab/CMakeLists.txt', 'python/CMakeLists.txt'

Are these changes tested?

Yes

github-actions[bot] commented 2 weeks ago

:warning: GitHub issue #41688 has been automatically assigned in GitHub to PR creator.

conbench-apache-arrow[bot] commented 2 weeks ago

After merging your PR, Conbench analyzed the 7 benchmarking runs that have been run so far on merge-commit 14b8ca53171435113a0f0f0c4ff1063d12543bc4.

There was 1 benchmark result indicating a performance regression:

The full Conbench report has more details.