Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[cmake] Change formatting of listfiles to use gersemi, remove cmake-format #2576

Draft
wants to merge 4 commits into
base: devel
Choose a base branch
from
Draft
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
58 changes: 0 additions & 58 deletions .cmake-format.yaml

This file was deleted.

12 changes: 12 additions & 0 deletions .gersemirc
Original file line number Diff line number Diff line change
@@ -0,0 +1,12 @@
definitions: [
./benchmark,
./bindings,
./CMakeLists.txt,
./cmake,
./src,
./unittest,
./utils
]
line_length: 100
indent: 2
warn_about_unknown_commands: false
3 changes: 3 additions & 0 deletions .git-blame-ignore-revs
Original file line number Diff line number Diff line change
Expand Up @@ -9,3 +9,6 @@

# pre-commit run -a (2024-09-04)
d7c57794672df89734a80c40c0bd3166c7a984c6

# pre-commit run -a (2025-02-12)
9f7b29a448498699de547ea84ce5f1e8f2a6e8ea
4 changes: 1 addition & 3 deletions .github/workflows/cmake/linux-debug-toolchain.cmake
Original file line number Diff line number Diff line change
@@ -1,6 +1,4 @@
# Don't generate debug symbols.
# This is useless for the CI build (we don't retrieve any binaries) and can stop the build
# by taking all the disk space.
set(CMAKE_CXX_FLAGS_DEBUG
""
CACHE STRING "")
set(CMAKE_CXX_FLAGS_DEBUG "" CACHE STRING "")
8 changes: 4 additions & 4 deletions .pre-commit-config.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,7 @@ ci:
autoupdate_branch: devel
autofix_prs: false
autoupdate_schedule: quarterly
submodules: true
repos:
- repo: https://github.com/pre-commit/mirrors-clang-format
rev: v19.1.6
Expand Down Expand Up @@ -31,11 +32,10 @@ repos:
hooks:
- id: ruff
- id: ruff-format
- repo: https://github.com/cheshirekow/cmake-format-precommit
rev: v0.6.13
- repo: https://github.com/BlankSpruce/gersemi
rev: 0.19.0
hooks:
- id: cmake-format
additional_dependencies: [pyyaml>=5.1]
- id: gersemi
exclude: pinocchiopy.pc.cmake
- repo: https://github.com/Lucas-C/pre-commit-hooks
rev: v1.5.5
Expand Down
Loading
Loading