Skip to content

Commit

Permalink
Run clang-tidy on headers (#996)
Browse files Browse the repository at this point in the history
This is a largely mechanical PR that extends our `clang-tidy` setup to
run on header files as well as implementations (an oversight in our
earlier introduction of the checker).

No new checks are added, so there shouldn't be anything controversial in
the changes here.

Fixes #995
  • Loading branch information
Baltoli authored Feb 27, 2024
1 parent 411cfc5 commit 84264eb
Show file tree
Hide file tree
Showing 33 changed files with 731 additions and 652 deletions.
3 changes: 3 additions & 0 deletions .clang-tidy
Original file line number Diff line number Diff line change
Expand Up @@ -21,6 +21,7 @@ Checks:
-bugprone-exception-escape
-bugprone-implicit-widening-of-multiplication-result
-bugprone-unchecked-optional-access
-bugprone-lambda-function-name
cppcoreguidelines-*
-cppcoreguidelines-avoid-magic-numbers
-cppcoreguidelines-avoid-non-const-global-variables
Expand All @@ -43,3 +44,5 @@ CheckOptions:
value: true
- key: 'cppcoreguidelines-macro-usage.CheckCapsOnly'
value: true
- key: 'cppcoreguidelines-special-member-functions.AllowSoleDefaultDtor'
value: true
Loading

0 comments on commit 84264eb

Please sign in to comment.