ci: use GitHub-hosted runners for public workflows - #30
Conversation
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
There was a problem hiding this comment.
🟢 Approval recommended
The runner migration is consistent across the affected workflows, and all Blacksmith-specific configuration in this PR is removed without leaving remaining references.
Pull request overview
This PR migrates CI verification and deployment workflows off Blacksmith and onto standard GitHub-hosted ubuntu-24.04 runners, aligning public workflows with GitHub’s default runner infrastructure while keeping the existing job structure/contracts.
Changes:
- Switch
runs-onfromblacksmith-2vcpu-ubuntu-2404toubuntu-24.04in verify, main, and release workflows. - Remove the Blacksmith self-hosted runner label allowlist from actionlint configuration.
File summaries
| File | Description |
|---|---|
| .github/workflows/verify.yml | Moves the verification job to GitHub-hosted ubuntu-24.04. |
| .github/workflows/release.yml | Runs verify/release jobs on GitHub-hosted ubuntu-24.04 (supports Trusted Publishing/provenance constraints). |
| .github/workflows/main.yml | Moves verify and deploy jobs to GitHub-hosted ubuntu-24.04. |
| .github/actionlint.yaml | Removes the Blacksmith runner label allowlist configuration. |
Review details
- Files reviewed: 4/4 changed files
- Comments generated: 0
- Review effort level: Lite
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Problem
This public repository still uses Blacksmith for verification and Worker deployment.
Solution
Use standard GitHub-hosted Ubuntu 24.04 for those jobs, preserving Linux x64 and the existing deploy/release contracts. Remove the unused Blacksmith runner allowlist.