-
Notifications
You must be signed in to change notification settings - Fork 158
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
Added .clang-format file and formatted everything #286
Conversation
Signed-off-by: Fabian Sauter <sauter.fabian@mailbox.org>
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.
Great contribution - thank you! I added comments on the files that we should look to skip, but other than that looks good
test/compiled_shaders_include/kompute_test/shaders/shadertest_logistic_regression.hpp
Outdated
Show resolved
Hide resolved
test/compiled_shaders_include/kompute_test/shaders/shadertest_op_custom_shader.hpp
Outdated
Show resolved
Hide resolved
Signed-off-by: Fabian Sauter <sauter.fabian@mailbox.org>
Those are things I could have thought about :D |
Nice one! Thanks for updating it so fast - just having a look. I can see that the format command now reflects the |
Signed-off-by: Fabian Sauter <sauter.fabian@mailbox.org>
I've used this script: https://github.com/eklitzke/clang-format-all/blob/master/clang-format-all |
Perfect - looks great. Nice one @COM8 , merging - thanks for the contribution |
This PR adds a
.clang-format
file based on the configuration specified in the Makefile.I also run it over all files in the repo once.