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

Add Clang-Tidy checks for case conventions, including our math conventions #905

Open
KipHamiltons opened this issue Jul 17, 2022 · 0 comments

Comments

@KipHamiltons
Copy link
Member

We could enforce naming conventions with built in clang-tidy checks.
We could also build custom checks for our codebase, including ones that enforce our math naming conventions.
An implementation of such checks might simply check the types of the tokens then make sure that if it's an eigen matrix, that it's named properly. You might even be able to make sure that compatible matrix multiplications are the only ones that happen. The options are endless 😅

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

No branches or pull requests

1 participant