Added initial .clang-tidy file - #2595
Conversation
|
Looks like it's already working in the linter :P |
No, that's clang-format detecting a file in the current master not being formatted correctly. I had the same linter error in my other PR. If we would add a GitHub action that checks clang-tidy, we'd see lots more errors :D |
|
I'm ok with making a big PR that fixes all the lint at once as well as adding other rules to improve code quality. |
|
And updating Github Action to reflect that. |
|
@Alexays I added the casing hints to the file, to show warnings in the IDE when the casing of the name of a class, function, struct, member variable, ... is inconsistent with the clang-tidy file. I based my choices on the existing code, to keep the impact to the code the lowest. One thing I'm unsure of: I added a Can you also check whether the other CheckOptions make sense to you? Let me know if you want to make any changes. I'll start tidying-up the code (in a separate commit) once you're sure about all the options. |
|
LGTM, we'll make small changes to the config if we realise it's too much. |
More info here: #2591