← Back to issue list

ci: skip tests on documentation-only PRs

View original Github issue

Metadata

Project
craft-parts
Number
#1647
Type
pull request
State
open
Author
Copilot
Labels
—
Created
Updated
Closed
—

Current evaluation

Adds CI path filtering to skip tests on documentation-only PRs, mirroring charmcraft's approach. Maintainer requested a test PR (#1648) to verify the filtering works.

Suggested action: keep open

Reason: Maintainer @lengau actively engaged by requesting a test PR (#1648) to verify the filtering mechanism, and Copilot created that follow-up PR. The PR has 10 unresolved review comments indicating ongoing review cycles. CI failures (OSV-scanner, java plugins on noble/s390x and ppc64el, lint/files) appear to be pre-existing infrastructure issues rather than caused by this PR's changes.

Impact: 40 Quick Win: 30.0 Staleness: 35 Complexity: 25 Confidence: 70

Issue body

Documentation-only PRs (e.g. #1645) were running the full test matrix unnecessarily. This adds the same path-filtering mechanism used in `canonical/charmcraft`. ## Changes - **New `filter` job** — uses `dorny/paths-filter@v4.0.2` to set `source-files=false` when all changed files match `docs/**`, `*.md`, `**/*.md`, or `.readthedocs.yaml` - **All `lint` and `test-*` jobs** — now `needs: filter` and forward `source-files` to the starflow reusable workflows, which skip test execution when the flag is `false` - **`merge_group` trigger** added to align with charmcraft

Evaluation history

Date Model Scores Action Summary
qwen3.6-35b-a3b-mtp-q6
Impact: 40
Quick Win: 30.0
Staleness: 35
Complexity: 25
Confidence: 70
keep open Adds CI path filtering to skip tests on documentation-only PRs, mirroring charmcraft's approach. Maintainer requested a test PR (#1648) to verify the filtering works.
qwen/qwen3.6-35b-a3b
Staleness: 10
Complexity: 15
Confidence: 85
needs review Adds CI path filtering to skip tests for documentation-only changes. Currently pending maintainer review with 10 unresolved comments and some failing CI checks. A follow-up PR was opened to test the filtering.
qwen3.6-35b-a3b-mtp-q6
Staleness: 5
Complexity: 15
Confidence: 80
keep open Adds CI path filtering to skip tests on documentation-only PRs. Awaiting maintainer review of a follow-up test PR to verify the filtering logic works as intended.

Update history

No update history recorded yet.

Related work

  • Related To: craft-parts#1648 (confidence 95%)

    Test PR created against #1647's branch to verify the docs-only filtering works correctly, as requested by maintainer @lengau.

  • Related To: starbase#545 (confidence 60%)

    Similar CI docs-only filtering feature in starbase project.

  • Related To: imagecraft#352 (confidence 60%)

    Similar CI docs-only filtering feature in imagecraft project.

Related issues

Issue Project State Summary Similarity
#1648 [WIP] ci: skip tests on documentation-only PRs craft-parts open Empty WIP test PR with 0 changes, created to verify CI filtering for docs-only PRs (#1647). Inactive 51 days with failing CI checks.
75%