Skip to content

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

Draft
blacksmith-sh[bot] wants to merge 5 commits intomainfrom
blacksmith-migration-60ca7c8
Draft

.github/workflows: Migrate workflows to Blacksmith runners#81414
blacksmith-sh[bot] wants to merge 5 commits intomainfrom
blacksmith-migration-60ca7c8

Conversation

@blacksmith-sh
Copy link

@blacksmith-sh blacksmith-sh bot commented Feb 4, 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.

@blacksmith-sh blacksmith-sh bot requested a review from a team as a code owner February 4, 2026 15:09
@github-actions
Copy link
Contributor

github-actions bot commented Feb 4, 2026


Thank you for your submission, we really appreciate it. Like many open-source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution. You can sign the CLA by just posting a Pull Request Comment same as the below format.


I have read the CLA Document and I hereby sign the CLA


1 out of 2 committers have signed the CLA.
✅ (coleaeason)[https://github.com/coleaeason]
@blacksmith-sh[bot]
You can retrigger this bot by commenting recheck in this Pull Request. Posted by the CLA Assistant Lite bot.

@melvin-bot melvin-bot bot requested review from MonilBhavsar and removed request for a team February 4, 2026 15:10
@melvin-bot
Copy link

melvin-bot bot commented Feb 4, 2026

@MonilBhavsar Please copy/paste the Reviewer Checklist from here into a new comment on this PR and complete it. If you have the K2 extension, you can simply click: [this button]

@coleaeason coleaeason removed the request for review from MonilBhavsar February 4, 2026 15:10
@coleaeason coleaeason self-assigned this Feb 4, 2026
@coleaeason coleaeason marked this pull request as draft February 4, 2026 15:11
@coleaeason coleaeason self-requested a review February 4, 2026 15:12
@blacksmith-sh
Copy link
Author

blacksmith-sh bot commented Feb 4, 2026

Found 65 test failures on Blacksmith runners:

Failures

Test View Logs
tests/actions/OnyxUpdateManagerTest.ts/
should pause SequentialQueue while missing updates are being fetched
View Logs
tests/actions/OnyxUpdateManagerTest.ts/
should trigger 2 GetMissingOnyxUpdates calls, because the deferred updates have gaps
View Logs
tests/actions/OnyxUpdateManagerTest.ts/should trigger Onyx update gap handling View Logs
tests/ui/BaseListItemTest.tsx/hover should work correctly View Logs
tests/ui/components/DisplayNames.test.tsx/
HTML is parsed by default when shouldParseFullTitle is not provided
View Logs
tests/ui/components/DisplayNames.test.tsx/
should parse HTML when shouldParseFullTitle is true
View Logs
tests/ui/components/IOURequestStepReportTest.tsx/
should set the reportID for all transactions when changing the report by the report fie
ld.
View Logs
tests/ui/components/LHNOptionsListTest.tsx/
does not show context menu when screen is not focused
View Logs
tests/ui/components/LHNOptionsListTest.tsx/
does not show queued message when user submits online with DEW policy
View Logs
tests/ui/components/LHNOptionsListTest.tsx/shows context menu after returning from chat View Logs
tests/ui/components/LHNOptionsListTest.tsx/shows context menu on long press View Logs
tests/ui/components/LHNOptionsListTest.tsx/
shows queued message when offline with pending DEW submit
View Logs
tests/unit/APITest.ts/
All write requests are in the queue should be called even some of them are the same
View Logs
tests/unit/APITest.ts/
different write APIs with resolveDuplicationConflictAction conflict
View Logs
tests/unit/APITest.ts/
duplicated write APIs with resolveDuplicationConflictAction conflict
View Logs
tests/unit/APITest.ts/
duplicated write APIs with resolveEnableFeatureConflicts conflict and same policyID
View Logs
tests/unit/APITest.ts/
multiple write APIs with resolveEnableFeatureConflicts conflict with different policyID
s
View Logs
tests/unit/APITest.ts/
multiple write APIs with resolveEnableFeatureConflicts conflict with some different pol
icyIDs
View Logs
tests/unit/APITest.ts/
Read request should not stuck when SequentialQueue is paused and resumed
View Logs
tests/unit/APITest.ts/Sequential queue will not run until credentials are read View Logs
tests/unit/APITest.ts/
Sequential queue will succeed if triggered while reauthentication via main queue is in
progress
View Logs
tests/unit/APITest.ts/
several actions made while offline will get added in the order they are created
View Logs
tests/unit/APITest.ts/
several actions made while offline will get added in the order they are created when we
need to reauthenticate
View Logs
tests/unit/APITest.ts/Write request can trigger reauthentication for anything retryable View Logs
tests/unit/APITest.ts/
Write request will move directly to the SequentialQueue when we are online and block no
n-Write requests
View Logs
tests/unit/APITest.ts/
Write request with supportal insufficient permissions (411) suppresses retry, applies f
ailureData, and triggers modal
View Logs
tests/unit/APITest.ts/write requests are retried when Auth is down View Logs
tests/unit/CLITests.ts/CLITests View Logs
tests/unit/IOUUtilsTest.ts/
should navigate to confirmation step with PAY iouType when iouType is SEND and not from
ManualDistanceRequest
View Logs
tests/unit/IOUUtilsTest.ts/
should navigate to confirmation step with provided iouType directly when shouldNavigate
ToSubmit = false in default case
View Logs
tests/unit/IOUUtilsTest.ts/
should navigate to confirmation step with reportIDParam if provided in default case
View Logs
tests/unit/IOUUtilsTest.ts/
should navigate to confirmation step with SEND iouType when iouType is SEND and from Ma
nualDistanceRequest
View Logs
tests/unit/IOUUtilsTest.ts/
should navigate to confirmation step with SUBMIT iouType when iouType is REQUEST
View Logs
tests/unit/IOUUtilsTest.ts/
should navigate to confirmation step with SUBMIT iouType when shouldNavigateToSubmit =
true in default case
View Logs
tests/unit/loginTest.tsx/renders correctly View Logs
tests/unit/QuickActionUtilsTest.ts/
Policy with per diem › should allow per diem action when policy has per diem rates
View Logs
tests/unit/ReportPrimaryActionUtilsTest.ts/
should return Approve for report being processed
View Logs
tests/unit/ReportPrimaryActionUtilsTest.ts/
should return EXPORT TO ACCOUNTING for finished reports
View Logs
tests/unit/ReportPrimaryActionUtilsTest.ts/
should return MARK AS CASH for broken connection
View Logs
tests/unit/ReportPrimaryActionUtilsTest.ts/
should return MARK AS CASH if has all RTER violations
View Logs
tests/unit/ReportPrimaryActionUtilsTest.ts/
should return PAY for expense report with payments enabled
View Logs
tests/unit/ReportPrimaryActionUtilsTest.ts/
should return REMOVE HOLD for an approver who held the expense
View Logs
tests/unit/ReportPrimaryActionUtilsTest.ts/
should return REMOVE HOLD for reports with transactions on hold
View Logs
tests/unit/ReportPrimaryActionUtilsTest.ts/
should return SUBMIT for expense report with manual submit
View Logs
tests/unit/ReportPrimaryActionUtilsTest.ts/
should return SUBMIT for open report in instant submit policy with no approvers
View Logs
tests/unit/Search/SearchQueryUtilsTest.ts/
returns query with all of the fields correctly substituted
View Logs
tests/unit/Search/SearchQueryUtilsTest.ts/returns query with updated groupBy View Logs
tests/unit/Search/SearchQueryUtilsTest.ts/returns query with updated view View Logs
tests/unit/Search/SearchQueryUtilsTest.ts/returns query with user emails substituted View Logs
tests/unit/Search/SearchQueryUtilsTest.ts/
returns query with user emails substituted and preserves user ids
View Logs
tests/unit/Search/SearchUIUtilsTest.ts/
Should create an optimistic parent report if the hasParentReport is false
View Logs
tests/unit/Search/SearchUIUtilsTest.ts/
Should create transaction thread report and set optimistic data necessary for its previ
ew
View Logs
tests/unit/Search/SearchUIUtilsTest.ts/
Should handle navigation if shouldNavigate = true
View Logs
tests/unit/SettlementButtonUtilsTest.ts/
match ROUTES.SEARCH_MONEY_REQUEST_REPORT over ROUTES.REPORT_WITH_ID
View Logs
tests/unit/SettlementButtonUtilsTest.ts/
navigate to ROUTES.MONEY_REQUEST_STEP_CONFIRMATION_VERIFY_ACCOUNT when active route is
ROUTES.MONEY_REQUEST_STEP_CONFIRMATION
View Logs
tests/unit/SettlementButtonUtilsTest.ts/
navigate to ROUTES.REPORT_VERIFY_ACCOUNT.getRoute(chatReportID) when active route is RO
UTES.REPORT_WITH_ID.getRoute(chatReportID)
View Logs
tests/unit/SettlementButtonUtilsTest.ts/
navigate to ROUTES.REPORT_VERIFY_ACCOUNT.getRoute(reportID) when active route is ROUTES
.REPORT_WITH_ID.getRoute(reportID)
View Logs
tests/unit/SettlementButtonUtilsTest.ts/
navigate to ROUTES.SEARCH_MONEY_REQUEST_REPORT_VERIFY_ACCOUNT when active route is ROUT
ES.SEARCH_MONEY_REQUEST_REPORT
View Logs
tests/unit/SettlementButtonUtilsTest.ts/
navigate to ROUTES.SEARCH_REPORT_VERIFY_ACCOUNT.getRoute(chatReportID) when active rout
e is ROUTES.SEARCH_REPORT.getRoute({reportID: chatReportID})
View Logs
tests/unit/SettlementButtonUtilsTest.ts/
navigate to ROUTES.SEARCH_REPORT_VERIFY_ACCOUNT.getRoute(reportID) when active route is
ROUTES.SEARCH_REPORT.getRoute(reportID)
View Logs
tests/unit/SettlementButtonUtilsTest.ts/
navigate to ROUTES.SEARCH_ROOT_VERIFY_ACCOUNT when active route is ROUTES.SEARCH_ROOT
View Logs
tests/unit/useNewTransactionsTest.ts/useNewTransactionsTest View Logs
tests/unit/useTransactionViolationsTest.ts/should filter out dismissed violations View Logs
tests/unit/useTransactionViolationsTest.ts/
should filter violations based on shouldShowViolation
View Logs
tests/unit/useTransactionViolationsTest.ts/
should respect shouldShowRterForSettledReport parameter
View Logs

Fix in Cursor

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.

1 participant