fix(ci): skip docs-only changes for Dependency Review, OSV Scanner - #658
fix(ci): skip docs-only changes for Dependency Review, OSV Scanner#658seonghobae wants to merge 1 commit into
Conversation
Org-wide audit found these workflows lack paths-ignore and fire full CI on every push/PR including docs-only changes, contributing to org-wide Actions queue congestion. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
|
Warning Review limit reachedNext included review available in 9 minutes. View limit detailsLimit details: You’ve used the included review currently available. You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. Review configuration: ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Team Run ID: 📒 Files selected for processing (2)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
| pull_request: | ||
| paths-ignore: | ||
| - "docs/**" | ||
| - "*.md" |
There was a problem hiding this comment.
🟡 Nested Markdown changes still launch scans
Markdown-only PRs under .jules or .agents still launch both scans because *.md matches only repository-root files.
Prompt for agents
Update the Markdown path-ignore pattern in both .github/workflows/dependency-review.yml and .github/workflows/osvscanner.yml so it matches Markdown files at any repository depth, including root files and nested files under .jules and .agents. Preserve the docs/** exclusion and verify the resulting GitHub Actions path-filter semantics for both root and nested Markdown-only pull requests.
Was this helpful? React with 👍 or 👎 to provide feedback.
|
Superseded by #663, which removes both central duplicates and applies the complete trigger-aware concurrency contract. |
What
Adds
paths-ignore: ["docs/**", "*.md"]to thepull_requesttrigger of.github/workflows/dependency-review.ymland.github/workflows/osvscanner.yml.Why
An org-wide audit of ContextualWisdomLab repos found workflow files lacking
paths-ignore, causing full CI (dependency review, OSV scanning) to run on every PR/push including docs-only changes. This needlessly triggers jobs and contributes to org-wide GitHub Actions queue congestion ("대적" — a backlog of queued runs that keeps growing instead of shrinking) under the org's shared 60-concurrent-job plan ceiling.Safety verification performed
branches/develop/protection) and both active rulesets ondevelop.osv-scan) appears in the classic required status check contexts (unit-and-api,cloud-e2e,Analyze (javascript-typescript),Analyze (python),property fuzz), so a docs-only PR that now skips these workflows will not starve a required check.CWL Central required workflowsruleset (id 18156473), which only pins workflows sourced from the central.githubrepo.paths-ignore/pathsfilter (audit-safe list confirmed live, not stale).pull_requestandworkflow_dispatchtriggers (nopushtrigger exists in either), sopaths-ignorewas added only underpull_requestper the fix scope.python3 -c "import yaml; yaml.safe_load(open(...))"after editing.No behavior change for code-touching PRs; docs/markdown-only PRs will simply no longer trigger these two workflows.
🤖 Generated with Claude Code