Adding clang tidy details

Henry Fredrick Schreiner
1 parent 101c926d
Showing 3 changed files with 39 additions and 6 deletions
.clang-tidy
CMakeLists.txt
examples/CMakeLists.txt
+#Checks: '*,-clang-analyzer-alpha.*'
+Checks: '-*,google-readability-casting,llvm-namespace-comment,performance-unnecessary-value-param,llvm-include-order,misc-throw-by-value-catch-by-reference,readability-container-size-empty,google-runtime-references,modernize*'
+HeaderFilterRegex: '.*hpp'
+CheckOptions:
+- key:             readability-braces-around-statements.ShortStatementLines
+  value:           '1'
+
@@ -22,6 +22,24 @@ else()
 endif()
+if(CMAKE_VERSION VERSION_GREATER 3.6)
+# Add clang-tidy if available
+    option(CLANG_TIDY_FIX "Perform fixes for Clang-Tidy (resets to OFF)" OFF)
+    find_program(
+        CLANG_TIDY_EXE
+        NAMES "clang-tidy"
+        DOC "Path to clang-tidy executable"
+    )
+
+    if(CLANG_TIDY_EXE)
+        if(CLANG_TIDY_FIX)
+            set(DO_CLANG_TIDY "${CLANG_TIDY_EXE}" "-fix")
+        else()
+            set(DO_CLANG_TIDY "${CLANG_TIDY_EXE}")
+        endif()
+    endif()
+endif()
+
 if(CMAKE_BUILD_TYPE STREQUAL Coverage)
     include(CodeCoverage)
@@ -64,3 +82,4 @@ if(CLI_EXAMPLES)
     add_subdirectory(examples)
 endif()
+set(CLANG_TIDY_FIX OFF CACHE BOOL "Perform fixes for Clang-Tidy (resets to OFF)" FORCE)
-add_executable(try try.cpp ${CLI_headers})
-target_link_libraries(try PUBLIC CLI11)
+function(add_cli_exe T)
+    add_executable(${T} ${ARGN} ${CLI_headers})
+    target_link_libraries(${T} PUBLIC CLI11)
-add_executable(try1 try1.cpp ${CLI_headers})
-target_link_libraries(try1 PUBLIC CLI11)
+    if(CLANG_TIDY_EXE)
+    set_target_properties(
+         ${T} PROPERTIES
+            CXX_CLANG_TIDY "${DO_CLANG_TIDY}"
+        )
+    endif()
+endfunction()
-add_executable(try2 try2.cpp ${CLI_headers})
-target_link_libraries(try2 PUBLIC CLI11)
+add_cli_exe(try try.cpp)
+add_cli_exe(try1 try1.cpp)
+add_cli_exe(try2 try2.cpp)