Skip to content

.github/workflows: Migrate workflows to Blacksmith runners#24

Open
useblacksmith-dev[bot] wants to merge 6 commits intomainfrom
blacksmith-migration-b2f08d6
Open

.github/workflows: Migrate workflows to Blacksmith runners#24
useblacksmith-dev[bot] wants to merge 6 commits intomainfrom
blacksmith-migration-b2f08d6

Conversation

@useblacksmith-dev
Copy link
Copy Markdown

@useblacksmith-dev useblacksmith-dev bot commented Apr 8, 2026

To whomever may be reviewing this PR,

Blacksmith is the fastest way to run your GitHub Actions.

What does this PR change?

This PR has been automatically generated by a team member in your GitHub organization using Blacksmith's Migration Wizard, or MigWiz for short. This PR changes the following:

  1. Your selected workflows will now run on Blacksmith's 2x faster hardware (e.g., runs-on: blacksmith-4vcpu-ubuntu-2204). Learn more about the different instances available to choose from.
  2. Your jobs running on Blacksmith will now have all official GitHub and popular third-party cache actions automatically interact with our 4x faster, colocated cache. Learn more about Blacksmith's actions cache.
  3. Your GitHub Actions will now actually be observable. Learn more about Blacksmith's logging and other observability features.
  4. Your Docker builds will now automatically share their Docker layer cache, resulting in up to 40x faster builds. Learn more about Blacksmith's Docker layer caching.

FAQ

  • Is this free? The first 3,000 minutes per month are free.
  • Who uses Blacksmith? Clerk, Ashby, VEED, and 600+ others.
  • What's the catch? There is none. Merge this thing already.

View in Codesmith
Codesmith can help with this PR — just tag @codesmith or enable autofix.

  • Autofix CI and bot reviews

View in Codesmith
Codesmith can help with this PR — just tag @codesmith or enable autofix.

  • Autofix CI and bot reviews (Staging)

Copy link
Copy Markdown

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 8567b6f. Configure here.

strategy:
matrix:
runs-on: [ubuntu-latest, windows-latest, macos-latest]
runs-on: [ubuntu-latest, blacksmith-4vcpu-windows-2025, macos-latest]
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Artifact name mismatch breaks verify and delete jobs

High Severity

The upload job's runs-on matrix update for Windows changed the artifact name. The verify and delete jobs still reference the old Windows artifact name, causing the verify job to fail and the delete job to attempt deleting a nonexistent artifact.

Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit 8567b6f. Configure here.

@blacksmith-sh
Copy link
Copy Markdown

blacksmith-sh bot commented Apr 8, 2026

Found 15 test failures on Blacksmith runners:

All test failures in the http-client package assert that the echoed URL from postman-echo.com starts with "http://" but receive "https://", indicating the HTTP client now enforces HTTPS for these requests. This change is not connected to the PR migrating workflows to Blacksmith runners, as the failures occur in local test runs where the client behavior has been altered to use secure connections by default.

Failures

Test View Logs
auth/does basic http get request with pat token auth
View Logs
auth/does basic http get request with pat token auth
View Logs
packages/http-client/tests/auth.test.ts/
does basic http get request with basic auth
View Logs
packages/http-client/tests/auth.test.ts/
does basic http get request with pat token auth
View Logs
packages/http-client/tests/basics.test.ts/does basic http get request
View Logs
packages/http-client/tests/basics.test.ts/
does basic http get request with default headers
View Logs
packages/http-client/tests/basics.test.ts/
does basic http get request with merged headers
View Logs
packages/http-client/tests/basics.test.ts/
does basic http get request with no user agent
View Logs
packages/http-client/tests/basics.test.ts/does basic http patch request
View Logs
packages/http-client/tests/basics.test.ts/does basic http post request
View Logs
packages/http-client/tests/keepalive.test.ts/
does basic http get request with keepAlive true
View Logs
packages/http-client/tests/keepalive.test.ts/
does basic http patch request with keepAlive true
View Logs
packages/http-client/tests/keepalive.test.ts/
does basic http post request with keepAlive true
View Logs
packages/http-client/tests/proxy.test.ts/
HttpClient does basic http get request through proxy
View Logs
packages/http-client/tests/proxy.test.ts/
HttpClient does basic http get request when bypass proxy
View Logs

Fix with Codesmith

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.

0 participants