docs: add filter to qa workflow
Metadata
Current evaluation
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.
Suggested action: —
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 |
|---|---|---|---|---|
| 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 issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #1212 ci: add path filter to qa workflow, make jobs conditional | rockcraft | merged | Merged. Implemented a path filter in the QA workflow to conditionally execute lint and test jobs based on modified files. Verified to skip execution for unrelated changes. Relies on upstream Starflow workflow updates. | |
| #2701 ci: filter in qa workflow | charmcraft | merged | Merged. Adds a filter to the QA workflow and passes it to the lint workflow. Approved by two reviewers. Tests are intentionally skipped to be verified before merge. | |
| #160 ci: add path filter to qa workflow, make jobs conditional | debcraft | merged | Merged: added a path filter to the QA workflow to conditionally run lint and test jobs. Approved, passed CI, and verified to skip for docs-only changes. Resolves CRAFT-5161. | |
| #6256 ci: add path filter to qa workflow, make jobs conditional | snapcraft | merged | Merged. Added a path filter to the QA workflow to conditionally run lint and test jobs based on changed files, resolving CRAFT-5161. Depends on upstream Starflow PR #143 and was verified to skip unrelated jobs. | |
| #2710 ci: fix qa workflow conditional | charmcraft | merged | Merged a fix correcting a typo in the CI QA workflow conditional that was locking the pipeline. Approved by two reviewers and passed CI checks. Restores workflow execution, with full resolution pending upstream starflow PR #146. | |
| #1218 ci: add checkout step to qa workflow filter | rockcraft | merged | Merged a fix for the QA workflow failing on push events by adding a required checkout step to the paths-filter action. Approved and merged after CI checks passed. | |
| #359 ci: use hosted runners for QA | craft-cli | merged | Merged update to CI configuration switching QA workflows from self-hosted to GitHub-hosted runners. Enables non-x86 architecture testing. Approved by two reviewers with all CI checks passing. Changes modify one file. | |
| #1570 ci: split QA workflows by runner | craft-parts | open | Splits QA workflow into separate GitHub-hosted and self-hosted runner files. 117 days old, zero review activity, 11 CI checks failing on self-hosted platforms (s390x, ppc64el, resolute, arm64). Not yet merged; qa.yaml in repo still unsplit. | |
| #1206 ci: make workflows conditional, add docs-only spread tests | rockcraft | merged | Merged after approval. Updates CI workflows to run conditionally on changed files, splitting source and documentation spread tests into separate runners. Predictable initial failures were acknowledged, and the changes were merged. | |
| #557 ci: set read permissions for qa workflow | starbase | merged | Merged updates granting explicit read permissions to the QA workflow. This allows internal projects to run GitHub API-dependent actions like paths-filter. Approved by two reviewers with all CI checks passing. |