ci: add path filter to qa workflow, make jobs conditional
Metadata
Current evaluation
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.
Suggested action: —
No scores available.
Issue body
Add the filter to the workflow directly, and pass it on to the Starflow lint and test workflows.
Depends on https://github.com/canonical/starflow/pull/143. The lint workflow has already been changed upstream.
For CRAFT-5161.
---
- [x] I've followed the [contribution guidelines](https://github.com/canonical/rockcraft/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. 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. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged. Implements path filtering in the QA workflow to conditionally execute lint and test jobs based on changed files. Author verified jobs correctly skip for documentation-only updates, optimizing CI for CRAFT-5161. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #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. | |
| #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. | |
| #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. | |
| #546 docs: add filter to qa workflow | starbase | merged | 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. | |
| #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. |