← Back to issue list

[WIP] ci: skip QA and Spread workflows for security-scanner-only PRs

View original Github issue

Metadata

Project
craft-application
Number
#1132
Type
pull request
State
closed
Author
Copilot
Labels
—
Created
Updated
Closed

Current evaluation

Draft follow-up PR #1132, opened only to test the paths-ignore change in parent PR #1131, was closed unmerged by mr-cal on 2026-09-10 as 'no longer relevant' once #1131 landed (commit d8885203).

Resolution: closed superseded

Reason: #1132 stacked on #1131 and existed solely to verify that change with a throwaway diff (+6 lines, 2 files). @mr-cal closed it on 2026-09-10 with 'No longer relevant, #1131 has landed'; #1131 had already merged as commit d8885203, adding paths-ignore for .github/workflows/policy.yaml and .osv-scanner.toml to the pull_request triggers in qa.yaml and spread.yaml.

Evaluated by: mr-cal-bot

No scores available.

Issue body

Thanks for the feedback on #1131. I've created this new PR, which merges into #1131, to address your comment. I will work on the changes and keep this PR's description up to date as I make progress. Original PR: #1131 Triggering comment (https://github.com/canonical/craft-application/pull/1131#issuecomment-4983351546): > @copilot make another draft PR that actually tests this

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed superseded Draft follow-up PR #1132, opened only to test the paths-ignore change in parent PR #1131, was closed unmerged by mr-cal on 2026-09-10 as 'no longer relevant' once #1131 landed (commit d8885203).
stealth/space-bunny-alpha — closed superseded Closed as superseded: draft PR #1132 was unnecessary after parent PR #1131 landed with the QA and Spread paths-ignore filters.
qwen/qwen3.6-35b-a3b — — Superseded and closed because the original PR #1131 has already landed. This draft PR was created to address feedback on #1131 but became unnecessary once the parent PR was merged.
qwen3.6-35b-a3b-mtp-q6
Impact: 5
Quick Win: 4
Staleness: 95
Complexity: 10
Confidence: 95
close stale Draft PR to merge into #1131 addressing feedback on security-scanner-only CI skip. #1131 is already merged, making this PR obsolete.
qwen/qwen3.6-35b-a3b
Staleness: 40
Complexity: 10
Confidence: 85
keep open Draft PR to skip QA and Spread CI workflows for security-scanner-only changes. Currently inactive, failing title policy check, and awaiting author progress.
qwen/qwen3.6-35b-a3b
Staleness: 40
Complexity: 10
Confidence: 80
keep open Draft PR modifying CI workflows to skip QA and Spread checks for security-scanner-only changes. Currently marked WIP, awaiting author progress, with minor CI failures on commit policy and OSV-scanner.
qwen/qwen3.6-35b-a3b
Staleness: 10
Complexity: 15
Confidence: 95
keep open Draft PR to skip QA and Spread workflows for security-scanner-only PRs. Currently contains zero changes, marked WIP, and author indicates development is ongoing. Awaiting further updates before review.
qwen3.6-35b-a3b-mtp-q6
Staleness: 5
Complexity: 35
Confidence: 90
keep open Draft CI workflow modification to skip QA and Spread jobs for security-scanner-only pull requests. Currently a work-in-progress draft merging into #1131, awaiting author implementation and testing.

Update history

Date Change
closed
updated

Related work

  • Superseded By: canonical/craft-application#1131 (confidence 96%)

    Parent/base PR that #1132 was created to test; it merged as commit d8885203 with the QA and Spread paths-ignore filters, making the test PR redundant.

  • Related To: canonical/craft-application#1130 (confidence 62%)

    Earlier merged PR (commit 235a5cc0) fixing the OSV-scanner workflow path and excluding docs-only inputs; #1131/#1132 follow up on that work by skipping QA and Spread runs for scanner-only PRs.

Related issues

Issue Project State Summary Similarity
#1131 ci: skip QA and Spread workflows for security-scanner-only PRs craft-application merged Merged in canonical/craft-application#1131 (commit d8885203), adding paths-ignore filters so QA and Spread workflows skip security-scanner-only PRs; test follow-up #1132 was closed as unnecessary.
87%
#600 ci: Add missing permissions to QA lint workflow starbase closed PR #600 was closed unmerged despite all CI checks passing; no merge commit or recorded replacement was found.
72%
#1648 [WIP] ci: skip tests on documentation-only PRs craft-parts closed Copilot's WIP docs-only test PR for the CI path filter in #1647 was closed after ~79 days with 0 files changed, 0 comments and failing checks. Parent #1647 is still open with unresolved review comments.
70%