-
Notifications
You must be signed in to change notification settings - Fork 91
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
cpplint for TravisCI builds #533
Comments
Strange decision. Google's style guide is very controversial :) |
There is need for some code style. And Google's one not the worst |
I thought we already had code style. What's the point reformatting all code for it? It's even documented in the wiki. Why not configure cpplint for existing rules? |
Should be a piece of cake with this Github Action: https://github.com/marketplace/actions/action-cpp-lint Alternatively, we could use a .editorconfig action since we already have .editorconfig |
As this project started to use https://github.com/marketplace/actions/clang-format-check |
clang-tidy has been added to workflow in #657 |
We are going to use https://github.com/cpplint/cpplint in order to make code compatible with http://google.github.io/styleguide/cppguide.html
The text was updated successfully, but these errors were encountered: