-
Notifications
You must be signed in to change notification settings - Fork 7
Add linting #34
Copy link
Copy link
Open
Labels
developmentFeatures/Tools related to development of GridKit, rather than use as a library.Features/Tools related to development of GridKit, rather than use as a library.enhancementNew feature or requestNew feature or request
Metadata
Metadata
Assignees
Labels
developmentFeatures/Tools related to development of GridKit, rather than use as a library.Features/Tools related to development of GridKit, rather than use as a library.enhancementNew feature or requestNew feature or request
There are a few style guidelines (#5) that can't be enforced by formatting alone, since they're outside the purview of formatting. For example: identifier naming schemes and error handling. These can be checked by a linter, though, and linting in general isn't a bad idea in a C++ codebase. If we're requiring
clang-formatas a dev dependency, then we can useclang-tidy, which is also included in clang. This can be added to precommit hooks (#33) and CI (#25).