docs: add filter to qa workflow
Metadata
Current evaluation
Merged PR (commit ccb17d2): adds a dorny/paths-filter "filter" job to .github/workflows/qa.yaml so lint/test reusable workflows only run for source files, skipping docs-only PRs.
Resolution: closed resolved
Reason: The PR itself was the fix and was merged to main as commit ccb17d2, adding a filter job that outputs a source-files path filter consumed by the lint and test reusable workflows. The reported fork hang was not a code bug; lengau explained the matrixed jobs were pending on a self-hosted runner tag unavailable to the fork owner.
Evaluated by: mr-cal-bot
No scores available.
Issue body
Add conditionals to skip required checks and unnecessary operations.
For a reason I don't understand right now, the test job is hanging on my fork: https://github.com/medubelko/starbase/pull/1. Normally I can test it just fine.
---
- [x] I've followed the [contribution guidelines](https://github.com/canonical/starbase/blob/main/CONTRIBUTING.md).
- [x] I've signed the [CLA](http://www.ubuntu.com/legal/contributors/).
- [x] I've successfully run `make lint && make test`.
- [ ] I've added or updated any relevant documentation.
- [ ] In documents I changed, I [added a meta description](https://canonical-starflow.readthedocs-hosted.com/how-to/add-a-page-meta-description/) if one was missing.
- [ ] I've updated the relevant release notes.
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged PR (commit ccb17d2): adds a dorny/paths-filter "filter" job to .github/workflows/qa.yaml so lint/test reusable workflows only run for source files, skipping docs-only PRs. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged in ccb17d2 (canonical/starbase#546), adding path filters to the QA workflow so required checks and unnecessary operations are skipped when irrelevant. | |
| qwen/qwen3.6-35b-a3b | — | — | Merged workflow updates adding conditionals to skip unnecessary documentation checks. The fork test job hanging was clarified as a missing self-hosted runner tag configuration issue. Approved by reviewers and merged after CI validation. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged. Added conditionals to skip required checks and unnecessary operations in the QA workflow. Fork test jobs stalled due to missing self-hosted runner tags, but the changes were reviewed and merged. |
Update history
No update history recorded yet.
Related work
-
Related To:
canonical/starbase#272
(confidence 40%)
Commit a411e24 (PR #272) introduced the starflow CI reusable workflows that this filter feeds source-files into.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #6256 ci: add path filter to qa workflow, make jobs conditional | snapcraft | merged | PR #6256 (CRAFT-5161) merged as commit 9a7074ce8, adding dorny/paths-filter to .github/workflows/qa.yaml, gating lint/test jobs on a paths filter and forwarding filters to Starflow workflows. | |
| #160 ci: add path filter to qa workflow, make jobs conditional | debcraft | merged | Merged in fa39c6f, adding QA path filtering and making Starflow lint/test jobs conditional so docs-only changes can skip them (CRAFT-5161). | |
| #1212 ci: add path filter to qa workflow, make jobs conditional | rockcraft | merged | PR #1212 merged: added path filters to rockcraft's QA workflow, made lint/test jobs conditional and passed filters on to the Starflow lint/test workflows (CRAFT-5161). Author verified jobs skip on docs-only changes. | |
| #1218 ci: add checkout step to qa workflow filter | rockcraft | merged | Merged PR #1218 by medubelko (+4 lines) added an actions/checkout step before dorny/paths-filter in the QA workflow, fixing filter failures on push events; approved with 1 reviewer. | |
| #2701 ci: filter in qa workflow | charmcraft | merged | Merged charmcraft#2701, adding path filters to the QA workflow and forwarding them to the called lint workflow for CRAFT-5161. | |
| #1647 ci: skip tests on documentation-only PRs | craft-parts | open | Adds a dorny/paths-filter 'filter' job to skip the full test matrix on docs-only PRs, gating all lint/test jobs via a new source-files input to the starflow reusable workflows. Idle 78 days, 10 unresolved review comments, lint/files failing. | |
| #6438 ci: Add missing permissions to QA lint workflow | snapcraft | closed | PR closed unmerged by lengau as redundant: the QA workflow (.github/workflows/qa.yaml) already grants workflow-level contents/pull-requests read permissions on main, so job-level permissions in the lint job were unnecessary. | |
| #194 ci: Set workflow-level read permissions for QA workflow | debcraft | merged | PR #194 merged: added workflow-level read permissions (actions, contents, pull-requests) to .github/workflows/qa.yaml and removed lint job-level overrides, fixing Starflow reusable workflow compilation on PRs. |