← Back to issue list

ci: add path filter to qa workflow, make jobs conditional

View original Github issue

Metadata

Project
snapcraft
Number
#6256
Type
pull request
State
merged
Author
medubelko
Labels
Created
Updated
Closed

Current evaluation

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.

Suggested action:

No scores available.

Issue body

Add the filter to the workflow directly, and pass it on to the Starflow lint and test workflows. Depends on https://github.com/canonical/starflow/pull/143. The lint workflow has already been changed upstream. For CRAFT-5161. --- - [x] I've followed the [contribution guidelines](https://github.com/canonical/snapcraft/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. 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.

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.
95%
#160 ci: add path filter to qa workflow, make jobs conditional debcraft merged Merged: added a path filter to the QA workflow to conditionally run lint and test jobs. Approved, passed CI, and verified to skip for docs-only changes. Resolves CRAFT-5161.
93%
#2701 ci: filter in qa workflow charmcraft merged Merged. Adds a filter to the QA workflow and passes it to the lint workflow. Approved by two reviewers. Tests are intentionally skipped to be verified before merge.
76%
#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.
75%
#2710 ci: fix qa workflow conditional charmcraft merged 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.
73%
#1218 ci: add checkout step to qa workflow filter rockcraft merged Merged a fix for the QA workflow failing on push events by adding a required checkout step to the paths-filter action. Approved and merged after CI checks passed.
71%