ci: fix qa workflow conditional
Metadata
Current evaluation
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.
Suggested action: —
No scores available.
Issue body
Correct a typo that locked up the workflow.
Won't completely fix the workflows until https://github.com/canonical/starflow/pull/146 is merged.
---
- [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 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. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged a fix for a typo in the CI QA workflow conditional that was causing failures. The change resolves the immediate lockup, though full workflow stability depends on canonical/starflow#146. Linkcheck issues were bypassed. |
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. | |
| #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. | |
| #815 ci: fix publishing workflow | craft-application | merged | Merged a single-line fix to the CI publishing workflow. Approved by two reviewers with all continuous integration checks passing. | |
| #260 ci: use starflow renovate config | starbase | merged | Merged the starflow renovate configuration update, removing 220 lines of CI setup across two files. Approved by two reviewers and successfully integrated. | |
| #1412 Test qa.yaml with starflow PR #101 changes | craft-parts | open | Temporary cross-repo validation PR pinning qa.yaml starflow workflow refs to the @copilot/use-pristine-ubuntu-action branch (starflow PR #101) instead of @main. 268 days old, no activity in 266 days, failing CI; current qa.yaml already uses @main. | |
| #72 fix(ci): use main branch from starflow | imagecraft | merged | Merged a maintainer's CI fix to use the main branch from starflow. The change updated 2 files with a +2/-2 line diff. | |
| #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. | |
| #71 fix(ci): use main branch from starflow | imagecraft | merged | Merged a fix updating the CI configuration to use the main branch from starflow. The change modified two files with two additions and two deletions. | |
| #272 ci: use starflow CI actions | starbase | merged | Merged after three reviewers approved and CI passed. The update switches the CI pipeline to starflow actions, resolves CRAFT-3603, and adds upstream dependencies. All review feedback was addressed before integration. |