Skip to content

ci: move PR checks to GitHub-hosted runners - #42

Merged
Svaag merged 2 commits into
mainfrom
ci/hosted-pr-runners
Oct 15, 2026
Merged

Svaag merged 2 commits into
mainfrom
ci/hosted-pr-runners

Conversation

@Svaag

@Svaag Svaag commented Oct 15, 2026

Copy link
Copy Markdown
Contributor

Moves the PR-check jobs in: ci.yml pr-agent.yml from [self-hosted, linux, x64, hyrule-public-pr] to ubuntu-latest (free and parallel for public repos) to remove the shared single-runner queue.

Each job was checked: no internal hosts / IPv6-only infra, no self-hosted-only tools (setup-python added where the job relied on the runner's system Python), no secrets beyond GITHUB_TOKEN. pr-agent keeps its fork guard (same-repo PRs or OWNER/MEMBER/COLLABORATOR slash commands, pull_request not pull_request_target).

Stays self-hosted: request-promotion / deploy workflows (post-merge, use promotion app secrets).

Move ci.yml pr-agent.yml  off the single self-hosted hyrule-public-pr runner. These jobs
need no infra network access and no secrets beyond GITHUB_TOKEN (pr-agent's
OPENROUTER key stays limited to same-repo PRs and trusted slash commands
under pull_request). Post-merge promotion/deploy jobs stay self-hosted.
@github-actions

Copy link
Copy Markdown

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

⏱️ Estimated effort to review: 1 🔵⚪⚪⚪⚪
🏅 Score: 95
🧪 No relevant tests
🔒 No security concerns identified
⚡ No major issues detected

@github-actions

Copy link
Copy Markdown

PR Code Suggestions ✨

No code suggestions found for the PR.

@Svaag
Svaag merged commit 839a4d2 into main Oct 15, 2026
5 checks passed
@Svaag
Svaag deleted the ci/hosted-pr-runners branch October 15, 2026 01:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant