[PR #6627] [MERGED] ci: check-c: run all checks in parallel mode #6098

Closed
opened 2026-05-05 10:50:43 -06:00 by gitea-mirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/netblue30/firejail/pull/6627
Author: @kmk3
Created: 1/23/2025
Status: Merged
Merged: 1/25/2025
Merged by: @kmk3

Base: masterHead: ci-check-c-parallel


📝 Commits (1)

  • aed960a ci: check-c: run all checks in parallel mode

📊 Changes

1 file changed (+3 additions, -3 deletions)

View changed files

📝 .github/workflows/check-c.yml (+3 -3)

📄 Description

This considerably speeds up the execution of the relevant steps, most
noticeably on scan-build:

  • scan-build: 121s to 55s (-54.5%)
  • cppcheck: 51s to 30s (-41.2%)
  • cppcheck-old: 21s to 11s (-47.6%)

Note: The above uses the average time of 3 runs on master (commit
bd946e359 ("Merge pull request #6624 from
powerjungle/chore/update-security-md", 2025-01-21)) [1] [2] [3] and 3
runs on this commit[4] [5] [6].

Note: This considers only the steps themselves, as the total time of
each job varies a lot between runs, mostly due to the apt-get
update/install steps. For example, the same apt-get install command
took 3 seconds on one scan-build job run[7] and 132 seconds on another
scan-build job run[8].

Related commits:

  • 500d8f2d6 ("ci: run make in parallel where applicable", 2023-08-14) /
    PR #5960
  • 5b1bd33c7 ("build: use full paths on compile/link targets",
    2023-07-02) /
    PR #6158

[1] https://github.com/netblue30/firejail/actions/runs/12928036246
[2] https://github.com/netblue30/firejail/actions/runs/12928180264
[3] https://github.com/netblue30/firejail/actions/runs/12928238010
[4] https://github.com/netblue30/firejail/actions/runs/12927984482/attempts/2
[5] https://github.com/netblue30/firejail/actions/runs/12927984482/attempts/3
[6] https://github.com/netblue30/firejail/actions/runs/12927984482/attempts/4
[7] https://github.com/netblue30/firejail/actions/runs/12928180264/job/36054893393
[8] https://github.com/netblue30/firejail/actions/runs/12881736117/job/35912783635


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/netblue30/firejail/pull/6627 **Author:** [@kmk3](https://github.com/kmk3) **Created:** 1/23/2025 **Status:** ✅ Merged **Merged:** 1/25/2025 **Merged by:** [@kmk3](https://github.com/kmk3) **Base:** `master` ← **Head:** `ci-check-c-parallel` --- ### 📝 Commits (1) - [`aed960a`](https://github.com/netblue30/firejail/commit/aed960a59414a71613d11387764eb372122e37cd) ci: check-c: run all checks in parallel mode ### 📊 Changes **1 file changed** (+3 additions, -3 deletions) <details> <summary>View changed files</summary> 📝 `.github/workflows/check-c.yml` (+3 -3) </details> ### 📄 Description This considerably speeds up the execution of the relevant steps, most noticeably on scan-build: * scan-build: 121s to 55s (-54.5%) * cppcheck: 51s to 30s (-41.2%) * cppcheck-old: 21s to 11s (-47.6%) Note: The above uses the average time of 3 runs on master (commit bd946e359 ("Merge pull request #6624 from powerjungle/chore/update-security-md", 2025-01-21)) [1] [2] [3] and 3 runs on this commit[4] [5] [6]. Note: This considers only the steps themselves, as the total time of each job varies a lot between runs, mostly due to the apt-get update/install steps. For example, the same `apt-get install` command took 3 seconds on one scan-build job run[7] and 132 seconds on another scan-build job run[8]. Related commits: * 500d8f2d6 ("ci: run make in parallel where applicable", 2023-08-14) / PR #5960 * 5b1bd33c7 ("build: use full paths on compile/link targets", 2023-07-02) / PR #6158 [1] https://github.com/netblue30/firejail/actions/runs/12928036246 [2] https://github.com/netblue30/firejail/actions/runs/12928180264 [3] https://github.com/netblue30/firejail/actions/runs/12928238010 [4] https://github.com/netblue30/firejail/actions/runs/12927984482/attempts/2 [5] https://github.com/netblue30/firejail/actions/runs/12927984482/attempts/3 [6] https://github.com/netblue30/firejail/actions/runs/12927984482/attempts/4 [7] https://github.com/netblue30/firejail/actions/runs/12928180264/job/36054893393 [8] https://github.com/netblue30/firejail/actions/runs/12881736117/job/35912783635 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
gitea-mirror 2026-05-05 10:50:43 -06:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: github-starred/firejail#6098
No description provided.