[PR #5960] [MERGED] ci: whitelist paths, reorganize workflows & speed-up tests #5755

New issue

Closed

opened 2026-05-05 10:44:18 -06:00 by gitea-mirror · 0 comments

gitea-mirror commented

2026-05-05 10:44:18 -06:00

Owner

📋 Pull Request Information

Original PR: https://github.com/netblue30/firejail/pull/5960
Author: @kmk3
Created: 8/16/2023
Status: ✅ Merged
Merged: 8/23/2023
Merged by: @kmk3

Base: master ← Head: ci-split-jobs

📝 Commits (10+)

b589045 ci: use path whitelists instead of blacklists
2f1b352 ci: rename profile-checks.yml to check-profiles.yml
5995a69 ci: trim comments in codeql-analysis.yml
8d53acb ci: move codeql python job into its own workflow
29f7a94 ci: remove "CI" from workflow names
82d2879 ci: split build and test into separate workflows
500d8f2 ci: run make in parallel where applicable
1c9af28 ci: move main code checks into new check-c.yml
5639359 docs: add missing CI badges to README.md
23a289a ci: split test jobs for faster checks

📊 Changes

10 files changed (+556 additions, -310 deletions)

View changed files

📝 .github/workflows/build-extra.yml (+29 -124)
📝 .github/workflows/build.yml (+40 -55)
➕ .github/workflows/check-c.yml (+161 -0)
📝 .github/workflows/check-profiles.yml (+7 -3)
➕ .github/workflows/check-python.yml (+55 -0)
➖ .github/workflows/codeql-analysis.yml (+0 -125)
📝 .github/workflows/codespell.yml (+2 -0)
➕ .github/workflows/test.yml (+253 -0)
📝 Makefile (+1 -0)
📝 README.md (+8 -3)

📄 Description

Main changes:

Run workflows based on path whitelists rather than blacklists
Reorganize jobs and workflows, mostly based on the paths used
Run make in parallel where applicable
docs: add missing CI badges to README.md
Split tests into multiple jobs

Considering the most recent runs, this reduces the total amount of time
it takes to run the tests from about 9-10 minutes to about 3 minutes.

_{🔄 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/5960 **Author:** [@kmk3](https://github.com/kmk3) **Created:** 8/16/2023 **Status:** ✅ Merged **Merged:** 8/23/2023 **Merged by:** [@kmk3](https://github.com/kmk3) **Base:** `master` ← **Head:** `ci-split-jobs` --- ### 📝 Commits (10+) - [`b589045`](https://github.com/netblue30/firejail/commit/b589045b0fde572b892b8fe9333e2148f6b28452) ci: use path whitelists instead of blacklists - [`2f1b352`](https://github.com/netblue30/firejail/commit/2f1b352e4e4fcc9c44fa618dfaf7e04b6f972caf) ci: rename profile-checks.yml to check-profiles.yml - [`5995a69`](https://github.com/netblue30/firejail/commit/5995a69e2c654b7dd6452acdabbfedd8510fd58a) ci: trim comments in codeql-analysis.yml - [`8d53acb`](https://github.com/netblue30/firejail/commit/8d53acbbb26928da60b215c71de90293ec70fbe6) ci: move codeql python job into its own workflow - [`29f7a94`](https://github.com/netblue30/firejail/commit/29f7a9461080871b83ecb2bfdce242a9266abbab) ci: remove "CI" from workflow names - [`82d2879`](https://github.com/netblue30/firejail/commit/82d28795a78fd0786fb833ebf7d3b4fe9c86d390) ci: split build and test into separate workflows - [`500d8f2`](https://github.com/netblue30/firejail/commit/500d8f2d6921f0d715d9e478e4fd10d6fe1f086f) ci: run make in parallel where applicable - [`1c9af28`](https://github.com/netblue30/firejail/commit/1c9af28611489dc3387cb44b20d0ab261b2053b0) ci: move main code checks into new check-c.yml - [`5639359`](https://github.com/netblue30/firejail/commit/5639359664827406c7bf12f3bc3dcc734119284c) docs: add missing CI badges to README.md - [`23a289a`](https://github.com/netblue30/firejail/commit/23a289a666b158714b9edde72d4a7dadef9ba6af) ci: split test jobs for faster checks ### 📊 Changes **10 files changed** (+556 additions, -310 deletions) <details> <summary>View changed files</summary> 📝 `.github/workflows/build-extra.yml` (+29 -124) 📝 `.github/workflows/build.yml` (+40 -55) ➕ `.github/workflows/check-c.yml` (+161 -0) 📝 `.github/workflows/check-profiles.yml` (+7 -3) ➕ `.github/workflows/check-python.yml` (+55 -0) ➖ `.github/workflows/codeql-analysis.yml` (+0 -125) 📝 `.github/workflows/codespell.yml` (+2 -0) ➕ `.github/workflows/test.yml` (+253 -0) 📝 `Makefile` (+1 -0) 📝 `README.md` (+8 -3) </details> ### 📄 Description Main changes: * Run workflows based on path whitelists rather than blacklists * Reorganize jobs and workflows, mostly based on the paths used * Run make in parallel where applicable * docs: add missing CI badges to README.md * Split tests into multiple jobs Considering the most recent runs, this reduces the total amount of time it takes to run the tests from about 9-10 minutes to about 3 minutes. --- <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:44:18 -06:00

closed this issue
added the
pull-request
label

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#5755

No description provided.

Rows
Columns