-
-
Notifications
You must be signed in to change notification settings - Fork 11
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
Basic Lint on .cpp files #228
Conversation
Is this using an open source tool that we can plug into our CI so that we do have to remember do this manually? Once we start linting & reformatting, it helps minimize code diffs to keep it up consistently, for which automation is the only real way to achieve that consistently (& without mental burden to boot). It looks like Google open sourced a tool |
|
PTAL soon. I've removed the util and capi files and revised the text. I'm using cpplint.py now. Clang_tidy at Google only works in google3 directories. I'll look into installing a public version locally. |
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. I will file an issue about getting clang-tidy to install locally & in CI
This doesn't include analysis of includes being used, nor names of variables.