Skip to content
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

.clang-tidy: Add basic .clang-tidy for identifier naming #1544

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

antangelo
Copy link
Contributor

Adds a .clang-tidy config that covers most identifier naming conventions. Ideally we can offload this component of PR reviews to CI along with formatting in #1510 .

Unfortunately it doesn't seem like clang-tidy supports having a separate option for global static variables vs global non-static variables (from what I can tell the StaticVariable option only applies to locals), so one limitation of this config is that it will recommend the g_ prefix for global static variables. The alternative is not to enforce a naming convention on the clang-tidy level, but we'd have to continue doing so manually in PR reviews.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant