ci: track inference + .github/e2e-track#166
Conversation
Updates e2e.yml to the latest template (auto-gate now infers track from .github/e2e-track on the PR base branch). Adds the file with 'main' so PRs into main auto-gate against the v0.5 stable matrix. The reusable-workflow uses: stays pinned at feat/version-matrix-phase1 until ci-core-e2e-runner#170 merges, then reverts to @main.
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (2)
📝 WalkthroughWalkthroughThe changes implement a dynamic track resolution system for E2E workflows, introducing a new Changes
Sequence DiagramsequenceDiagram
participant GitHub as GitHub Workflow Event
participant Parse as Parse Job
participant Comment as Comment Check
participant API as GitHub API<br/>(gh api)
participant File as Base Branch<br/>.github/e2e-track
participant Validate as Track Validation
participant Output as Workflow Output
GitHub->>Parse: Trigger with event data
Parse->>Comment: Check if override<br/>in comment
alt Comment Override Present
Comment->>Validate: Use explicit track
else No Comment Override
Comment->>API: Fetch .github/e2e-track<br/>from base branch
API->>File: Retrieve file
File-->>API: Return track value
alt File Exists & Readable
API-->>Validate: Decoded track value
else File Missing/Error
Comment->>Validate: Use 'main' fallback<br/>+ warning
end
end
Validate->>Validate: Test against<br/>TRACK_PATTERN allowlist
alt Track Valid
Validate->>Output: Emit resolved track
Output->>GitHub: Job succeeds
else Track Invalid
Validate->>GitHub: Fail job
end
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~25 minutes Poem
✨ Finishing Touches🧪 Generate unit tests (beta)
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 |
Updates the auto-gate to read the target track from
.github/e2e-trackon the PR's base branch, so PRs into different long-lived branches automatically use the right matrix.This branch's main path:
e2e.ymlto the latest template.github/e2e-track = mainso PRs into main auto-gate against v0.5 stable.Companion: v2-dev gets its own e2e-track = v0.6-dev (separate update).
Required before merge: ci-core-e2e-runner#170 (the matrix system itself). Until that merges, the reusable-workflow uses: stays pinned at the feature branch.
Summary by CodeRabbit