[clang-tidy] enforce header consideration

sarthou · Jun 9, 2024 · 1d6c78e · 1d6c78e
1 parent 38df470
commit 1d6c78e
Show file tree

Hide file tree

Showing 2 changed files with 21 additions and 39 deletions.
diff --git a/.clang-tidy b/.clang-tidy
@@ -74,44 +74,26 @@ Checks: >
   readability-simplify-subscript-expr,
   readability-string-compare,
   readability-use-std-min-max
-  
+WarningsAsErrors: "*"
+HeaderFilterRegex: 'include/*'
 CheckOptions:
-  - key: readability-identifier-naming.NamespaceCase,          
-    value: lower_case
-  - key: readability-identifier-naming.ClassCase,              
-    value: CamelCase 
-  - key: readability-identifier-naming.StructCase,             
-    value: CamelCase 
-  - key: readability-identifier-naming.StructSuffix,           
-    value: _t        
-  - key: readability-identifier-naming.FunctionCase,           
-    value: camelBack 
-  - key: readability-identifier-naming.VariableCase,           
-    value: lower_case
-  - key: readability-identifier-naming.PrivateMemberCase,      
-    value: lower_case
-  - key: readability-identifier-naming.PrivateMemberSuffix,    
-    value: _         
-  - key: readability-identifier-naming.EnumCase,               
-    value: CamelCase 
-  - key: readability-identifier-naming.EnumSuffix,             
-    value: _e        
-  - key: readability-identifier-naming.EnumConstantCase,       
-    value: lower_case
-  - key: readability-identifier-naming.GlobalConstantCase,     
-    value: UPPER_CASE
-  - key: readability-identifier-naming.StaticConstantCase,     
-    value: UPPER_CASE
-  - key: readability-identifier-naming.StaticVariableCase,     
-    value: lower_case
-  - key: readability-identifier-naming.StaticVariableSuffix,   
-    value: _         
-  - key: readability-operators-representation.BinaryOperators, 
-    value: "&&;&=;&;|;~;!;!=;||;|=;^;^="
-  - key: misc-const-correctness.AnalyzeValues,                 
-    value: false     
-  - key: misc-include-cleaner.DeduplicateFindings,             
-    value: false
-
+  - { key: readability-identifier-naming.NamespaceCase,         value: lower_case }
+  - { key: readability-identifier-naming.ClassCase,             value: CamelCase }
+  - { key: readability-identifier-naming.StructCase,            value: CamelCase }
+  - { key: readability-identifier-naming.StructSuffix,          value: _t }
+  - { key: readability-identifier-naming.FunctionCase,          value: camelBack }
+  - { key: readability-identifier-naming.VariableCase,          value: lower_case }
+  - { key: readability-identifier-naming.PrivateMemberCase,     value: lower_case }
+  - { key: readability-identifier-naming.PrivateMemberSuffix,   value: _ }
+  - { key: readability-identifier-naming.EnumCase,              value: CamelCase }
+  - { key: readability-identifier-naming.EnumSuffix,            value: _e }
+  - { key: readability-identifier-naming.EnumConstantCase,      value: lower_case }
+  - { key: readability-identifier-naming.GlobalConstantCase,    value: UPPER_CASE }
+  - { key: readability-identifier-naming.StaticConstantCase,    value: UPPER_CASE }
+  - { key: readability-identifier-naming.StaticVariableCase,    value: lower_case }
+  - { key: readability-identifier-naming.StaticVariableSuffix,  value: _ }
+  - { key: readability-operators-representation.BinaryOperators, value: '&&;&=;&;|;~;!;!=;||;|=;^;^=' }
+  - { key: misc-const-correctness.AnalyzeValues,                value: false }
+  - { key: misc-include-cleaner.DeduplicateFindings,            value: false }
 ExtraArgsBefore:
   - '-frelaxed-template-template-args'
diff --git a/.github/run_clang_tidy.py b/.github/run_clang_tidy.py
@@ -35,7 +35,7 @@
 
 def runclangtidy(filepath):
     print("Checking: " + filepath)
-    proc = subprocess.Popen(f'clang-tidy --quiet -warnings-as-errors=* -p {builddir} {filepath}', shell=True)
+    proc = subprocess.Popen(f'clang-tidy -warnings-as-errors=* -p {builddir} {filepath}', shell=True)
     if proc.wait() != 0:
         failedfiles.append(filepath)