(MOT-4277) fix(harness): bootstrap benchmark history branch - #638
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (3)
📝 WalkthroughWalkthroughA new script initializes a missing remote ChangesBenchmark history initialization
Estimated code review effort: 3 (Moderate) | ~20 minutes Sequence Diagram(s)sequenceDiagram
participant GitHubActions
participant BootstrapScript
participant GitRemote
GitHubActions->>BootstrapScript: Run bootstrap_benchmark_history.py
BootstrapScript->>GitRemote: Check refs/heads/gh-pages
GitRemote-->>BootstrapScript: Return branch status
BootstrapScript->>GitRemote: Push empty commit if missing
GitRemote-->>BootstrapScript: Confirm branch creation
GitHubActions->>GitRemote: Checkout gh-pages
Possibly related PRs
Poem
✨ Finishing Touches📝 Generate docstrings
🧪 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 |
skill-check — worker0 verified, 49 skipped (no docs/).
Four for four. Nicely done. |
Refs MOT-4277
Summary
gh-pagesbranch beforebenchmark-actionstores the first metricsRoot cause
The first publisher run successfully downloaded the compact metrics and full execution artifacts, but
benchmark-actiontried to fetchgh-pagesbefore that branch existed. The job stopped before reaching the workflow's existing orphan-branch fallback, so no history or Pages artifact was published.Validation
python3 -m py_compile .github/scripts/bootstrap_benchmark_history.pygit diff --checkSummary by CodeRabbit
New Features
Bug Fixes
Tests