-
Notifications
You must be signed in to change notification settings - Fork 3
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
build: introduce pre-commit #18
Conversation
0d1efff
to
c607f9c
Compare
@fvranicTT Do we maybe want to block binaries from being committed as well? Are they blocked by default somehow? If so, is the filter configurable, for instance if we want to add documentation in the future and some .png sneaks in as a screenshot instead of a .svg. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks
A very good question. I tried blocking binaries with |
45740b2
to
93f8971
Compare
This reverts commit 7e81cb9.
Description:
This PR introduces consistent formatting for all C++ and Python files to ensure uniform code style across the repository. While this is not a final set of formatting rules, it serves as an initial baseline that can be refined over time.
Since this change affects most files, merging it will help keep future PRs clean and focused on functional changes rather than formatting inconsistencies. After merging, I will follow up with another PR to remove this change from git history, preventing unnecessary git blame noise.
Next Steps:
Let me know if you have any concerns or suggestions!