From 2f4cb79ef2a8539e6d2ed52b4e6bec62de144c2b Mon Sep 17 00:00:00 2001 From: Aleksey Loginov Date: Fri, 16 Feb 2024 18:39:34 +0300 Subject: [PATCH 1/2] extend clang-tidy --- .clang-tidy | 8 ++++++-- .pre-commit-config.yaml | 4 ++-- CMakePresets.json | 2 +- 3 files changed, 9 insertions(+), 5 deletions(-) diff --git a/.clang-tidy b/.clang-tidy index 3b7221329..c8f8f1c39 100644 --- a/.clang-tidy +++ b/.clang-tidy @@ -1,10 +1,14 @@ --- Checks: 'clang-diagnostic-*,clang-analyzer-*,-*,bugprone-*,concurrency-*,performance-*,-macro*,readability-identifier-naming,-bugprone-exception-escape' -ExtraArgs: ['-std=c++20'] WarningsAsErrors: '*' HeaderFilterRegex: './src/.*' AnalyzeTemporaryDtors: false FormatStyle: 'file' +HeaderFileExtensions: + - h + - hpp +ImplementationFileExtensions: + - cpp CheckOptions: - { key: readability-identifier-naming.NamespaceCase, value: lower_case } - { key: readability-identifier-naming.ClassCase, value: lower_case } @@ -17,4 +21,4 @@ CheckOptions: - { key: readability-identifier-naming.TemplateParameterCase, value: CamelCase } - { key: readability-identifier-naming.TypeTemplateParameterCase, value: CamelCase } - { key: readability-identifier-naming.MacroDefinitionCase, value: UPPER_CASE } - # - { key: readability-identifier-naming.TypeAliasCase, value: lower_case } \ No newline at end of file + # - { key: readability-identifier-naming.TypeAliasCase, value: lower_case } diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml index 1f2ce803c..bf227f36b 100644 --- a/.pre-commit-config.yaml +++ b/.pre-commit-config.yaml @@ -38,6 +38,6 @@ repos: # rev: master # hooks: # - id: clang-tidy -# args: [--fix-errors, -config-file=.clang-tidy, --checks=-readability-identifier-naming*] -# files: src/.* +# args: [--fix-errors, -config-file=.clang-tidy, "--checks=-*,bugprone-*,concurrency-*,performance-*", -p=build, ./src/*] +# # files: src # entry: /usr/local/opt/llvm/bin/clang-tidy diff --git a/CMakePresets.json b/CMakePresets.json index 9cb4bc8b5..aedd3afd7 100644 --- a/CMakePresets.json +++ b/CMakePresets.json @@ -81,7 +81,7 @@ "name": "clang-tidy", "hidden": true, "cacheVariables": { - "CMAKE_CXX_CLANG_TIDY": "clang-tidy;-header-filter=${sourceDir}/src/.*;-checks=-*,bugprone-*,concurrency-*,performance-*,-macro*,-bugprone-exception-escape;-warnings-as-errors=*;-extra-arg=-std=c++20" + "CMAKE_CXX_CLANG_TIDY": "clang-tidy;-p=${sourceDir}/build;-header-filter=${sourceDir}/src/.*;-checks=-*,bugprone-*,concurrency-*,performance-*,-macro*,-bugprone-exception-escape;-warnings-as-errors=*" } }, { From d11ab311120d646cd5ea2980fb50d04a6e11cfbb Mon Sep 17 00:00:00 2001 From: Aleksey Loginov Date: Fri, 16 Feb 2024 18:42:50 +0300 Subject: [PATCH 2/2] Update CMakePresets.json --- CMakePresets.json | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/CMakePresets.json b/CMakePresets.json index aedd3afd7..576a5507c 100644 --- a/CMakePresets.json +++ b/CMakePresets.json @@ -81,7 +81,7 @@ "name": "clang-tidy", "hidden": true, "cacheVariables": { - "CMAKE_CXX_CLANG_TIDY": "clang-tidy;-p=${sourceDir}/build;-header-filter=${sourceDir}/src/.*;-checks=-*,bugprone-*,concurrency-*,performance-*,-macro*,-bugprone-exception-escape;-warnings-as-errors=*" + "CMAKE_CXX_CLANG_TIDY": "clang-tidy;-header-filter=${sourceDir}/src/.*;-checks=-*,bugprone-*,concurrency-*,performance-*,-macro*,-bugprone-exception-escape;-warnings-as-errors=*" } }, { @@ -303,4 +303,4 @@ } } ] -} \ No newline at end of file +}