ci: add conditionals for docs-only checks
Metadata
Current evaluation
Merged pull request adding CI conditionals to skip checks for documentation-only changes. Approved by a reviewer, passed all CI checks, and successfully integrated into the main branch.
Suggested action: —
No scores available.
Issue body
Add conditionals to skip required checks and unnecessary operations.
For CRAFT-5161.
Tested in my fork: https://github.com/medubelko/imagecraft/pull/2
---
- [x] I've followed the [contribution guidelines](https://github.com/canonical/imagecraft/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.
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Merged pull request adding CI conditionals to skip checks for documentation-only changes. Approved by a reviewer, passed all CI checks, and successfully integrated into the main branch. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged. Implements CI conditionals to skip required checks and unnecessary operations for documentation-only changes, resolving CRAFT-5161. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #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. | |
| #1560 ci: use travis conditionals | snapcraft | merged | Merged pull request updating CI configuration to use Travis conditionals. The change modified one file (+20/-10 lines) and received approval from one reviewer before being integrated. | |
| #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. | |
| #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. | |
| #18 ci: ensure tests succeed in CI | craft-application | merged | Merged updates to CI configuration ensuring tests pass successfully. Approved by one reviewer, modifying five files with 23 additions and 13 deletions. | |
| #17 ci: update docs CI | craft-application | merged | Documentation CI configuration updated and merged to resolve Sphinx linting failures. Approved by one reviewer with passing CI checks. Changes span four files with 17 additions and 5 deletions. | |
| #66 ci: add documentation build CI step | craft-archives | merged | Merged into main following approval and passing CI checks. The change introduces a new CI step to automate documentation builds across two files. Code coverage remains stable at 88.07%. | |
| #143 ci: make test jobs conditional | starflow | merged | Merged to conditionally enable CI test jobs, satisfying Charmcraft dependencies and upcoming Terraform ruleset requirements. Approved by one reviewer, passed all CI checks, and updated one file. |