Skip to content

Conversation

@hendrikmuhs
Copy link
Contributor

@hendrikmuhs hendrikmuhs commented Oct 5, 2025

replace the custom check style script with pre-commit. Fixes the long-existing problem of clang-format inconsistencies, by installing a fixed version of clang-format from pip.

@hendrikmuhs hendrikmuhs marked this pull request as ready for review October 7, 2025 06:02
@hendrikmuhs
Copy link
Contributor Author

eventually this should run on all files. However at the moment I see too many cpplint errors to be fixed.

@hendrikmuhs
Copy link
Contributor Author

hendrikmuhs commented Oct 9, 2025

Next steps:

  • hooks for python, rust
  • eventually run hooks on all files
  • enable check on push

Discarded

@hendrikmuhs hendrikmuhs merged commit 1269361 into KeyviDev:master Oct 14, 2025
17 checks passed
@hendrikmuhs hendrikmuhs deleted the pre-commit branch October 14, 2025 19:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant