Skip to content

Commit 4b52def

Browse files
committed
ci: run make sparse as part of the GitHub workflow
Occasionally we receive reviews after patches were integrated, where `sparse` (https://sparse.docs.kernel.org/en/latest/ has more information on that project) identified problems such as file-local variables or functions being declared as global. By running `sparse` as part of our Continuous Integration, we can catch such things much earlier. Even better: developers who activated GitHub Actions on their forks can catch such issues before even sending their patches to the Git mailing list. This addresses gitgitgadget#345 Signed-off-by: Johannes Schindelin <[email protected]>
1 parent 75ae10b commit 4b52def

File tree

1 file changed

+8
-0
lines changed

1 file changed

+8
-0
lines changed

.github/workflows/main.yml

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -350,6 +350,14 @@ jobs:
350350
- uses: actions/checkout@v1
351351
- run: ci/install-dependencies.sh
352352
- run: ci/run-static-analysis.sh
353+
sparse:
354+
runs-on: ubuntu-20.04
355+
steps:
356+
- name: Install `sparse` and other dependencies
357+
run: |
358+
sudo apt-get install -q -y sparse libssl-dev libcurl4-openssl-dev libexpat-dev gettext zlib1g-dev
359+
- uses: actions/checkout@v2
360+
- run: make sparse
353361
documentation:
354362
needs: ci-config
355363
if: needs.ci-config.outputs.enabled == 'yes'

0 commit comments

Comments
 (0)