Skip to content

Faster pre-commit #445

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.

Sign up for GitHub

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

Merged
MaciejKaszynski merged 2 commits into from
Aug 11, 2026
+6 −11
Merged

Faster pre-commit #445

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
3657133
Remove unnecessary clang format hook
danth Aug 10, 2026
fb22ad3
Use faster copyright fix hook
danth Aug 10, 2026
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
17 changes: 6 additions & 11 deletions .pre-commit-config.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -14,25 +14,15 @@
repos:
- repo: local
hooks:
- id: copyright-fix
name: //:copyright-fix
entry: bazel run //:copyright-fix
language: system
pass_filenames: false
- id: format-fix
name: //:format.fix
entry: bazel run //:format.fix
language: system
files: (BUILD|BUILD\.bazel|WORKSPACE|\.bazelrc|[^/]+\.(py|bzl|bazel|yaml|yml|rs))$
files: (BUILD|BUILD\.bazel|WORKSPACE|\.bazelrc|[^/]+\.(py|bzl|bazel|yaml|yml|rs|c|cpp|h|hpp))$
pass_filenames: false
- repo: https://github.com/pocc/pre-commit-hooks
rev: v1.3.5
hooks:
- id: clang-format
name: clang-format
entry: bazel run //:format.fix_C++_with_clang-format
language: system
files: '\.(c|cc|cpp|h|hpp)$'
- id: clang-tidy
name: clang-tidy
# Run through a wrapper that sanitizes unsupported linker flags in
Expand All @@ -49,3 +39,8 @@ repos:
tests/|
score/
)
- repo: https://github.com/eclipse-score/tools
rev: 7694bf823421b5542cd2f45239b8bdf205dbface
hooks:
- id: copyright
exclude: ^.github/
Loading