CI: move cppcheck job to CI_formatting file

Minor tweak, put the non-test-case-oriented CI actions into a separate workflow file just so it is easier to keep track of and expand in the future if more small checks / tools are integrated. Signed-off-by: Eric Richter <[email protected]>
erichte-ibm · Aug 23, 2023 · 9a64db8 · 9a64db8
1 parent 2e3bb2c
commit 9a64db8
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 18 deletions.
diff --git a/.github/workflows/CI_formatting.yml b/.github/workflows/CI_formatting.yml
@@ -1,9 +1,9 @@
-name: Test Source Code Formatting
+name: Format Check and Static Analysis
 
 on: ['push', 'pull_request']
 
 jobs:
-  build:
+  formatcheck:
 
     runs-on: ubuntu-latest
 
@@ -15,3 +15,18 @@ jobs:
 
     - name: test if formatting is appropriate
       run: make formatcheck
+
+  cppcheck:
+
+    runs-on: ubuntu-latest
+
+    steps:
+    - uses: actions/checkout@v3
+      with:
+        submodules: 'recursive'
+
+    - name: install cppcheck
+      run: sudo apt-get update && sudo apt-get install -y cppcheck
+
+    - name: run cppcheck
+      run: make cppcheck
diff --git a/.github/workflows/x86_CI.yml b/.github/workflows/x86_CI.yml
@@ -38,19 +38,3 @@ jobs:
 
     - name: run test cases
       run: make check SECVAR_TOOL=$(pwd)/build/secvarctl
-
-  cppcheck:
-
-    runs-on: ubuntu-latest
-
-    steps:
-    - uses: actions/checkout@v3
-      with:
-        submodules: 'recursive'
-
-    - name: install cppcheck
-      run: sudo apt-get update && sudo apt-get install -y cppcheck
-
-    - name: run cppcheck
-      run: make cppcheck
-