-
Notifications
You must be signed in to change notification settings - Fork 400
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
Add proper clang-format to pre-commit #1514
Conversation
files: '.*\.(cpp|h|hpp|cu|cuh)$' | ||
language: system | ||
- id: clang-format | ||
types_or: [c++, c, c#, cuda] # also run on CUDA |
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.
c# ? 🥳
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.
It's supported, so why not. Main issue is that the OG hook doesn't support CUDA as a type, although I am working on getting it fixed.
Other supported languages include JSON btw.
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.
The CUDA type_or is now merged in, but won't be updated until a new release is made of the hook.
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.
LGTM
…into upgrade-clang-format-hook
* Add proper clang-format to pre-commit * Change mirror * Autoupdate hooks and freeze clang-format * Try to fix cpp-lint * Remove cpp_lint job * Rollback from RC * Add comment
Motivation and Context
How Has This Been Tested
Types of changes
Checklist