feat(runs-on-selector): add runs-on selector action - #10
Conversation
|
Navigate logical layers of code changes, visualize relationships, and explore their blast radius. Warning Review limit reachedNext included review available in 47 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: Advanced Run ID: 📒 Files selected for processing (1)
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Advanced Run ID: 📒 Files selected for processing (12)
Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review. WalkthroughAdds a Node 24 GitHub Action that selects a runner from pull-request labels and a caller-provided pool map. CI resolves the runner before dependent jobs. Tests, documentation, Node tooling, and release metadata are included. ChangesRunner selection
Priority: ⬇️ Low Estimated code review effort: 3 (Moderate) | ~25 minutes Change: Feature Sequence Diagram(s)sequenceDiagram
participant CI
participant RunnerSelector
participant GitHubEvent
participant DependentJobs
CI->>RunnerSelector: Provide pool map and default pool
RunnerSelector->>GitHubEvent: Read pull-request labels
GitHubEvent-->>RunnerSelector: Return event labels
RunnerSelector-->>CI: Write runs-on output
CI->>DependentJobs: Schedule check and test jobs on selected runner
Merge Risk: ⚪ Minimal · up to The action selects runners from pull-request labels, validates configuration, and fails safely on invalid selections. No merge-blocking production risk remains evident. 🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
Full details: Docstring CoverageExplanation Docstring coverage is 22.22% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 18 functions across 5 files. (7 skipped: 7 unsupported.) ✨ Finishing Touches 💡 1📝 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. A rabbit maps the pools with care Comment |
b10195f to
2c8ab2b
Compare
194593a to
4ae7b69
Compare
Signed-off-by: Yordis Prieto <yordis.prieto@gmail.com>
4ae7b69 to
6752709
Compare
A runner label nobody provisioned queues instead of failing, so a job reading the resolved value needs a ceiling. Signed-off-by: Yordis Prieto <yordis.prieto@gmail.com>
runs-onin that pull request, so the choice arrived as a change to the workflow rather than as a decision about where to run it.Need help on this PR? Tag
@codesmith-botwith what you need. Autofix is disabled.