ci: add checkout step to qa workflow filter
Metadata
Current evaluation
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.
Suggested action: —
No scores available.
Issue body
Fixes the QA workflow failing on `push` events.
---
- [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 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. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged a fix adding a checkout step to the QA workflow filter. This resolves failures on push events, as the paths-filter action requires checked-out code for push triggers rather than pull requests. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #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. | |
| #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. | |
| #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. | |
| #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. | |
| #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. | |
| #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. |