ci: make workflows conditional, add docs-only spread tests
Metadata
Current evaluation
Closed without merging. The changes to conditional CI workflows and docs-only Spread tests remained pending review with unresolved comments and failing snap-tests. The PR was likely abandoned or superseded.
Suggested action: —
No scores available.
Issue body
Make workflows conditional:
- The full lint job only runs if source files are changed.
- The Spread and test workflows only run if source files are changed.
- The Spread workflow only runs the `test/**` Spread tests.
Add a docs Spread workflow that runs the `docs/**` Spread tests.
With the QA workflow, if there's a change to the source, all linters run in one runner (`make lint`). But for the Spread tests, there's a hard split between the docs and source Spread tests, meaning each will spawn their own runner. This isn't bad, but it adds complexity and inconsistency between the different workflow types. If Starflow's workflows were completely split, we could properly partition the checks in both collections (QA source, QA docs, Spread source, Spread docs).
---
- [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 | — | — | Closed without merging. The changes to conditional CI workflows and docs-only Spread tests remained pending review with unresolved comments and failing snap-tests. The PR was likely abandoned or superseded. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged changes to condition CI workflows on file changes and add a docs-only Spread test workflow. Source and docs tests now run in separate runners, improving consistency and reducing unnecessary CI execution. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged CI updates to conditionally run workflows based on modified files. Split Spread tests into separate source and docs pipelines, adding a dedicated docs-only workflow to improve consistency and reduce runner overhead. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #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. | |
| #6245 ci: make workflows conditional, add docs-only spread tests | snapcraft | merged | Merged. Configured CI workflows to run conditionally based on changed files, separating source and documentation pipelines. Added a docs-only spread test workflow. Approved and merged. | |
| #545 ci: add conditionals for docs-only checks | starbase | closed | Closed without merging due to pending review and failing CI tests. The changes to skip unnecessary checks for documentation-only updates were abandoned after 49 days with unresolved comments and failed jammy test runs. | |
| #1219 ci: run full spread workflow on scheduled suite | rockcraft | closed | Closed without merging. The change to run full Spread tests on scheduled CI runs failed the source-spread-tests check and received no reviews. Likely abandoned due to the failing CI and lack of reviewer feedback. | |
| #6244 ci: test docs-conditional workflows and jobs | snapcraft | closed | Abandoned and closed without merging. The pull request failed integration tests on multiple Ubuntu versions and received no reviews or comments. | |
| #3480 ci: spread testing | snapcraft | closed | Closed without merging. The maintainer submitted the pull request as a temporary test, requested it be ignored, and abandoned it without review or CI checks. | |
| #3374 [wip] ci: migrate spread tests to github actions | snapcraft | closed | Marked as WIP, the pull request to migrate spread tests to GitHub Actions was closed without review or merge. No CI checks ran and no reviewers were assigned, indicating the work was abandoned. | |
| #813 ci: spread test on arm | craft-application | closed | Closed without merging. The PR adding ARM spread tests was abandoned due to failing snap-tests on arm64 and lack of reviewer engagement over several months. | |
| #3089 ci: move staging store tests to spread | snapcraft | closed | Closed without being merged. The pull request to move staging store tests to spread was abandoned, receiving no reviews, CI checks, or comments before closure. | |
| #2770 ci: always discard spread workers in snap-store-tests | charmcraft | closed | Abandoned without merge. The change to add a cleanup step preventing CI VM leaks in snap-store-tests received no reviews and was closed. |