Skip to content

ci: cut the per-push job fan-out that queues CI behind the runner limit - #1508

Merged
Makisuo merged 1 commit into
mainfrom
ci/trim-runner-fanout
Oct 15, 2026
Merged

Makisuo merged 1 commit into
mainfrom
ci/trim-runner-fanout

ci: cut the per-push job fan-out that queues CI behind the runner limit

5f64e26
Select commit
Loading
Failed to load commit list.
Maple Review Bot / Maple / review succeeded Oct 15, 2026 in 4m 15s

Confidence 9/10 · No issues found

🟢 Confidence 9/10 · safe to merge
Ran the new matrix jq for empty, one- and four-language change sets; the legs match the description and no other job behaviour changed.
quality 100/100 · no findings · tests not needed · risk low

Path-filters CodeQL's Rust/Go/Python/Actions legs on PRs and packs the Bun suites into the existing test lanes, to cut the per-push job fan-out. The workflow change reproduces the removed five-language matrix on every non-PR event, so it is safe to merge.

  • codeql.yml builds the analyze matrix from paths-filter's changes output
  • PRs analyze Rust/Go/Python/Actions only when their paths change
  • Push, schedule and dispatch keep the previous five-language matrix
  • Bun suites pack into the small test lanes via a second Turbo invocation

Before merge

  • Manual · Confirm no ruleset or merge protection requires an Analyze (...) check a skipping PR would leave pending
What was checked
  • Ran the matrix jq for [], ["python"], ["rust"] and all four: JS/TS is always emitted (codeql.yml:77-86)
  • Filters match real files: git ls-files counts 22 .rs, 14 .go, 8 .py (codeql.yml:52-66)
  • Non-PR events force exactly the five languages the removed static matrix ran (codeql.yml:74-99)

5f64e26 · Updated on every push. Reply "won't fix" to dismiss a finding, or mention @maple-review-bot to ask about one.