Skip to content

ci: use GitHub-hosted runners for public workflows - #30

Merged
altaywtf merged 1 commit into
mainfrom
ci/github-hosted-runners
Sep 5, 2026
Merged

ci: use GitHub-hosted runners for public workflows#30
altaywtf merged 1 commit into
mainfrom
ci/github-hosted-runners

Conversation

@altaywtf

@altaywtf altaywtf commented Sep 5, 2026

Copy link
Copy Markdown
Member

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.

Copilot AI lite review requested due to automatic review settings September 5, 2026 16:55
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Sep 5, 2026

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review Completed 2026-09-05T16:57:07.574332Z 1cd4196 PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@slopzapper slopzapper Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✨ CLEAN

Clean runner swap: public jobs now use the valid GitHub-hosted ubuntu-24.04 label, and the actionlint Blacksmith allowlist is gone because nothing still needs it.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟢 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-on from blacksmith-2vcpu-ubuntu-2404 to ubuntu-24.04 in 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.

@altaywtf
altaywtf merged commit 5a15a49 into main Sep 5, 2026
6 checks passed
@altaywtf
altaywtf deleted the ci/github-hosted-runners branch September 5, 2026 17:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants