Thyme icon indicating copy to clipboard operation
Thyme copied to clipboard

Apply clang tidy to class names

Open duncanspumpkin opened this issue 3 years ago • 2 comments

This only applies the following clang tidy check.

  • key: readability-identifier-naming.ClassCase value: 'CamelCase'

I will revert the other changes clang tidy made without asking.

duncanspumpkin avatar Nov 16 '21 07:11 duncanspumpkin

Could we do something similar to how clang-format is set up so we can run a format from a target and catch style violations on the CI. I'm happy to enforce more of the style guide automatically :D

OmniBlade avatar Nov 18 '21 10:11 OmniBlade

Yes that is possible. It will take a little while to get in the state where this could be enforced. This GitHub action could be used https://github.com/platisd/clang-tidy-pr-comments

duncanspumpkin avatar Nov 18 '21 11:11 duncanspumpkin