chore: switch CI runners to self-hosted#16
Conversation
|
Warning You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again! |
📝 WalkthroughWalkthroughThis PR introduces infrastructure changes to GitHub Actions workflows by switching two build job runners from cloud-hosted to self-hosted runners, adds a Lake build trace configuration file, and introduces a comprehensive new Lean formalization file implementing a Level-Based Theorem approach for analyzing r-local games. ChangesCI/CD Runner Migration
LBT Coupling Formalization
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Tip 💬 Introducing Slack Agent: The best way for teams to turn conversations into code.Slack Agent is built on CodeRabbit's deep understanding of your code, so your team can collaborate across the entire SDLC without losing context.
Built for teams:
One agent for your entire SDLC. Right inside Slack. Comment |
There was a problem hiding this comment.
Actionable comments posted: 2
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In @.github/workflows/blueprint.yml:
- Line 19: The "Install LaTeX" workflow step uses apt-get (Linux-only) but the
self-hosted runner is macOS, so update the workflow to run platform-aware
installation: change the "Install LaTeX" step to detect runner OS (using if:
matrix.os or if: runner.os == 'Linux' / runner.os == 'macOS') and run apt-get
only on Linux, and use brew install or a macOS-appropriate installer (or skip if
TeX is preinstalled) on macOS; alternatively replace the step with a
cross-platform action such as actions/setup-tex or add distinct self-hosted
labels (e.g., self-hosted, linux) in runs-on to ensure apt-get is only executed
on Linux runners.
In `@LBTCoupling.lean.bak`:
- Around line 1-6: The file LBTCoupling.lean.bak is a backup/temporary Lean
formalization accidentally committed; either remove it from the PR or rename it
to LBTCoupling.lean if it is intended to be part of the project, and ensure the
CI configuration (the sorry-check exclusion) and the three documented `sorry`
occurrences are accounted for when renaming; if this is a draft/backup, delete
LBTCoupling.lean.bak from the branch and open a separate PR for the
formalization instead.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: aeac7f74-3fe1-4893-91a0-ca22a7eded6a
⛔ Files ignored due to path filters (1)
.lake/config/[anonymous]/lakefile.olean.lockis excluded by!**/*.lock
📒 Files selected for processing (5)
.github/workflows/blueprint.yml.github/workflows/ci.yml.lake/config/[anonymous]/lakefile.olean.lake/config/[anonymous]/lakefile.olean.traceLBTCoupling.lean.bak
Switch all GitHub Actions workflows from GitHub-hosted runners (ubuntu-latest, macos-latest) to self-hosted runner to reduce GitHub runner minute consumption.
Summary by CodeRabbit