From 9a64db8dffac7440915d300854db22e2057921c5 Mon Sep 17 00:00:00 2001 From: Eric Richter Date: Wed, 23 Aug 2023 10:26:36 -0500 Subject: [PATCH] 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 --- .github/workflows/CI_formatting.yml | 19 +++++++++++++++++-- .github/workflows/x86_CI.yml | 16 ---------------- 2 files changed, 17 insertions(+), 18 deletions(-) diff --git a/.github/workflows/CI_formatting.yml b/.github/workflows/CI_formatting.yml index d3fa7da0..af3541bf 100644 --- 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 index 3574406f..4cd8fdf8 100644 --- 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 -