Merge pull request #158 from skrobinson/fix-SA-runner

Refactor configuration for StaticAnalysis

.github/workflows/static_analysis.yml

@@ -28,15 +28,13 @@ jobs:
       uses: actions/checkout@v2
 
     - name: Analyze
-      uses: JacobDomagala/StaticAnalysis@master
+      uses: skrobinson/StaticAnalysis@stable-for-argparse
       with:
         clang_tidy_args: >-
           --config-file=$GITHUB_WORKSPACE/.clang-tidy
           --extra-arg=-I$GITHUB_WORKSPACE/include --extra-arg=-std=c++17
         cppcheck_args: >-
           --enable=all --inconclusive --inline-suppr
-          -i$GITHUB_WORKSPACE/test/main.cpp
-          -i$GITHUB_WORKSPACE/test/test_*.cpp
           --suppress=missingInclude
           --suppress='*:$GITHUB_WORKSPACE/test/doctest.hpp'
         init_script: tools/static_analysis_setup.sh

test/CMakeLists.txt

@@ -57,10 +57,3 @@ set_property(TARGET ARGPARSE PROPERTY CXX_STANDARD 17)
 
 # Set ${PROJECT_NAME} as the startup project
 set_property(DIRECTORY ${CMAKE_CURRENT_SOURCE_DIR} PROPERTY VS_STARTUP_PROJECT ARGPARSE)
-
-file(GLOB ARGPARSE_LINT_SOURCES
-    tidy-base.cpp
-)
-ADD_EXECUTABLE(ARGPARSE_LINT ${ARGPARSE_LINT_SOURCES})
-set_target_properties(ARGPARSE_LINT PROPERTIES OUTPUT_NAME tidy-base)
-set_property(TARGET ARGPARSE_LINT PROPERTY CXX_STANDARD 17)

tools/CMakeLists.txt

@@ -0,0 +1,10 @@
+cmake_minimum_required(VERSION 3.12)
+project(argparse)
+
+file(GLOB ARGPARSE_LINT_SOURCES
+    tidy-base.cpp
+)
+ADD_EXECUTABLE(ARGPARSE_LINT ${ARGPARSE_LINT_SOURCES})
+set_target_properties(ARGPARSE_LINT PROPERTIES OUTPUT_NAME tidy-base)
+set_property(TARGET ARGPARSE_LINT PROPERTY CXX_STANDARD 17)
+INCLUDE_DIRECTORIES("../include" ".")

tools/static_analysis_setup.sh

@@ -1,4 +1,4 @@
 #!/bin/bash
 
-# Change to the "test" subdir before "build" subdir is made.
-cd test
+# Change to the "tools" subdir before "build" subdir is made.
+cd tools