Skip to content

perf(ci): run checks only for affected components - #121

Merged
GaganSD merged 3 commits into
mainfrom
ci/ownership-based-checks
Nov 6, 2026
Merged

GaganSD merged 3 commits into
mainfrom
ci/ownership-based-checks

Conversation

@GaganSD

@GaganSD GaganSD commented Nov 5, 2026 •

Copy link
Copy Markdown
Owner

Route tooling checks by ownership, with conservative fallback for unknown paths and a weekly full sweep.

Routing impact

  • Replayed both routing rules against the last 30 first-parent commits on main before this PR
Work across those 30 commits Old routing New routing Reduction
Pi package jobs 101 40 61 fewer · 60%
Herdr jobs, counting each Node version separately 90 39 51 fewer · 57%
Tooling jobs 30 9 21 fewer · 70%
Total test jobs 221 88 133 fewer · 60%
Post-main quickstart jobs¹ 30 16 14 fewer · 47%
  • ¹ Assumes successful validation and publication where applicable. Daily/manual quickstarts and the new weekly full sweep are excluded; mandatory routing, secret scanning, and final-gate jobs are not counted as test jobs.

@GaganSD
GaganSD marked this pull request as ready for review November 6, 2026 09:31
@greptile-apps

greptile-apps Bot commented Nov 6, 2026 •

Copy link
Copy Markdown

RetriggerView in GreptileConfidence Score: 5/5

[Medium impact] The PR appears safe to merge; no actionable defects were found.

Summary

This PR selects CI checks by file ownership instead of running unrelated suites.

  • CI runs checks only for components that use the changed paths.
  • Main runs Quickstart only for relevant changes and after publication.

No actionable defects were found. GaganSD’s stated policy intentionally skips post-main quickstart for Herdr-only and ordinary documentation changes.

Diagram

%%{init: {'theme': 'neutral'}}%%
flowchart TD
  A[PR or main push] --> B[Validate release intent and select checks]
  C[Manual or weekly run] --> B
  B --> D[Selected package, Herdr, and tooling checks]
  A --> E[Secret scan]
  C --> E
  B --> F[CI passed gate]
  D --> F
  E --> F
  F --> G{Main push?}
  G -->|Yes| H[Check release eligibility]
  H --> I[Publish or intentionally skip]
  I --> J{Quickstart selected?}
  J -->|Yes| K[Run quickstart on the same commit]
Loading

Reviews (1) · Last reviewed commit: "ci: remove unused routing wrapper and re..." · Reviewed by Greptile

@GaganSD
GaganSD merged commit b9007a9 into main Nov 6, 2026
14 checks passed
@GaganSD GaganSD changed the title ci: run checks only for affected components perf(ci): run checks only for affected components Nov 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant