ci: make non-ci workflows fork-friendly#239
Open
Bronek wants to merge 4 commits into
Open
Conversation
Fall back to GitHub-hosted runners and gate secret/Pages steps to the base repo so forks run on hosted infra. Target WarpBuild runners by warp-* label instead of runner group, pinning intel->x64-4x and arm64->arm64-2x. Assisted-by: Claude:claude-opus-4-8
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
🤖 Augment PR SummarySummary: Make CI workflows runnable in forks by falling back to GitHub-hosted runners and gating steps that require base-repo infrastructure/secrets. Changes:
Technical Notes: Uses 🤖 Was this summary useful? React with 👍 or 👎 |
![augmentcode[bot]](https://avatars.githubusercontent.com/in/1027498?s=60&v=4){kind=small}
Probe with 'brew list --versions' instead of 'brew --prefix' inside the test: brew --prefix on an uninstalled keg exits non-zero, which under the step's -eo pipefail shell could abort before the install branch ran. Assisted-by: Claude:claude-opus-4-8
Member
Author
|
augment review |
WarpBuild Windows runners are slow to start; use windows-2022 (VS 2022) and windows-2025-vs2026 (VS 2026) unconditionally. Drop the now-unused warp-windows-latest-x64-4x label. Assisted-by: Claude:claude-opus-4-8
Disable Homebrew auto-update/cleanup globally and bump the job timeout to 30 min so a fork's cold toolchain install doesn't intermittently hit the limit. Run an explicit 'brew update' only on the cold-install path so the requested formula version is found despite a stale runner index. Assisted-by: Claude:claude-opus-4-8
Bronek
force-pushed
the
bronek/fork_friendly
branch
from
628f660 to
15750a3
Compare
Member
Author
|
augment review |
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fall back to GitHub-hosted runners and gate secret/Pages steps to the base repo so forks run on hosted infra. Target WarpBuild runners by warp-* label instead of runner group, pinning intel->x64-4x and arm64->arm64-2x.