ci: filter in qa workflow
Metadata
Current evaluation
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.
Suggested action: —
No scores available.
Issue body
Add the filter to the workflow directly, and pass it on to the called lint workflow.
Depends on https://github.com/canonical/starflow/pull/141 and https://github.com/canonical/starflow/pull/143.
For CRAFT-5161.
---
- [x] I've followed the [contribution guidelines](https://github.com/canonical/charmcraft/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. 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. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged to address CRAFT-5161. Adds a filter to the QA workflow and passes it to the lint workflow. Reviewer noted this disables test enforcement but accepts it for pre-merge validation. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #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. | |
| #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. | |
| #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. | |
| #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. | |
| #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. | |
| #141 ci: remove filter from lint workflow | starflow | merged | Merged following two approvals and successful CI checks. Removes the filter from the lint workflow, delegating it to the calling workflow which defaults to affected source files. Resolves part of CRAFT-5161. | |
| #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. | |
| #1131 ci: skip QA and Spread workflows for security-scanner-only PRs | craft-application | merged | Merged. Added paths-ignore filters to qa.yaml and spread.yaml to skip QA and Spread CI pipelines for PRs only modifying security scanner configurations. Approved by two reviewers and successfully merged. |