ci: add conditionals for docs-only checks
Metadata
Current evaluation
Merged CRAFT-5161 CI changes in #352 (commit 0104193), adding conditionals to skip required checks and unnecessary operations for docs-only changes.
Resolution: closed resolved
Reason: PR #352 was approved and merged as commit 0104193. Its path conditionals reduce unnecessary CI work for documentation-only changes.
Evaluated by: mr-cal-bot
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 |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged CRAFT-5161 CI changes in #352 (commit 0104193), adding conditionals to skip required checks and unnecessary operations for docs-only changes. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged the CRAFT-5161 CI update to skip required checks and unnecessary operations for docs-only changes; successfully tested in medubelko/imagecraft#2. | |
| 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 work
-
Related To:
medubelko/imagecraft#2
(confidence 95%)
The author's fork PR was used to test the CI changes before upstream PR #352.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #545 ci: add conditionals for docs-only checks | starbase | closed | PR #545 was closed unmerged after 113 days; its docs-only CI filtering was delivered separately by merged PR #546 in commit ccb17d2. | |
| #1205 ci: make workflows conditional, add docs-only spread tests | rockcraft | closed | Superseded by merged PR canonical/rockcraft#1206 (commit 3624f64c), which implemented conditional CI and split source and docs Spread tests. | |
| #17 ci: update docs CI | craft-application | merged | Merged in craft-application#17 via commit f71a2aec, updating docs CI to fix Sphinx linting failures; remaining lint/test CI fixes were handled in #18. | |
| #489 ci: add CLA check (CRAFT-305) | charmcraft | merged | Merged PR #489 added a CI CLA check for CRAFT-305 in commit 6ed6bec1; merge commit: 81a87192. | |
| #1206 ci: make workflows conditional, add docs-only spread tests | rockcraft | merged | Merged in canonical/rockcraft#1206 at commit 3624f64c, adding path-conditional CI and separate source and docs-only Spread workflows. |