Skip to content

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

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

ci: use GitHub-hosted runners for public workflows#129
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's verification and release workflows select paid Blacksmith runners.

Solution

Move eight Linux jobs to standard GitHub-hosted Ubuntu 24.04 x64 and four native Darwin jobs to macOS 26 ARM64. Keep the existing verification, signing/notarization, triggers, permissions, action pins and release Environments. Remove the obsolete Blacksmith-only actionlint allowance.

The macOS mapping preserves the current OS and architecture: Blacksmith latest is macOS 26 ARM64, and GitHub offers standard macos-26 ARM64 runners.

Proof

Actionlint 1.7.12 and offline zizmor 1.29.0 passed (no findings; two existing first-party-ref suppressions). A comparison against main confirms the workflow files changed only at the twelve runner labels; the deleted linter config contained no other policy. Native verification will run in the existing PR lane. Signing/notarization and publication were not manually invoked.

Copilot AI lite review requested due to automatic review settings September 5, 2026 16:47

@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

Runner labels are valid GitHub-hosted images, actionlint 1.7.12 already accepts macos-26, and no leftover Blacksmith labels remain in-repo.

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 label updates are consistent across workflows, and Blacksmith-specific configuration has been fully removed without impacting workflow structure.

Pull request overview

This PR updates the repository’s public GitHub Actions workflows to run exclusively on standard GitHub-hosted runners instead of paid Blacksmith runners, keeping the existing workflow structure (triggers, permissions, pins, and environments) intact.

Changes:

  • Replaced Blacksmith Ubuntu runner labels with ubuntu-24.04 across CI and release verification jobs.
  • Replaced Blacksmith macOS runner labels with macos-26 for native Darwin verification and release jobs.
  • Removed the now-obsolete .github/actionlint.yaml allowance that permitted blacksmith-* self-hosted runner labels.
File summaries
File Description
.github/workflows/release-slopmachine.yml Switches verify/release jobs to GitHub-hosted Ubuntu 24.04 and macOS 26 runners.
.github/workflows/release-slopguard.yml Switches verify, Darwin verify, and release jobs to GitHub-hosted Ubuntu 24.04 and macOS 26 runners.
.github/workflows/ci.yml Switches all CI jobs previously using Blacksmith runners to GitHub-hosted Ubuntu 24.04 and macOS 26 runners.
.github/actionlint.yaml Removes the Blacksmith-only runner-label allowance (file removed).
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.

@chatgpt-codex-connector

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:49:27.743837Z 26ebfdc 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.

@altaywtf
altaywtf merged commit b801fd5 into main Sep 5, 2026
12 checks passed
@altaywtf
altaywtf deleted the ci/github-hosted-runners branch September 5, 2026 16:54
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