-
Notifications
You must be signed in to change notification settings - Fork 404
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 tools/format.sh which formats all files automatically. #14
Conversation
Uses clang-format, cmake-format, and buildifier.
Ryan, can you try this script on your PR #13? Especially cmake-format and buildifier. :) |
@maxgolov what do you think? This kind of duplicates In particular, I'd like something more automated than |
I got this error when I ran on os x
|
Looks like BSD vs GNU sed. |
Also if clang-format-8 doesn't exist, fall back to clang-format.
Please try again. :) |
Worked! |
Thank you for your patience, and testing this on OSX which I didn't have access to. I think I need one more approval to merge. |
Uses clang-format, cmake-format, and buildifier.