Skip to content

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

Merged
Moeabdelaziz007 merged 1 commit into
mainfrom
blacksmith-migration-b4c49fb
May 14, 2026
Merged

.github/workflows: Migrate workflows to Blacksmith runners#72
Moeabdelaziz007 merged 1 commit into
mainfrom
blacksmith-migration-b4c49fb

Conversation

@blacksmith-sh
Copy link
Copy Markdown
Contributor

@blacksmith-sh blacksmith-sh Bot commented May 14, 2026

This PR has been automatically generated by a team member in your GitHub organization using Blacksmith's Migration Wizard. 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.

@Moeabdelaziz007 Moeabdelaziz007 merged commit 63d668f into main May 14, 2026
0 of 2 checks passed
@Moeabdelaziz007 Moeabdelaziz007 deleted the blacksmith-migration-b4c49fb branch May 14, 2026 07:24
@greptile-apps
Copy link
Copy Markdown

greptile-apps Bot commented May 14, 2026

Greptile Summary

This PR migrates the syntax-check job in python-package-conda.yml from ubuntu-latest to blacksmith-4vcpu-ubuntu-2404, matching the runner already in use by the e2e-signal-flow job in the same file.

  • The change is purely a runner label swap with no modifications to workflow logic, steps, or action versions.
  • The PR title uses a path-based prefix rather than the Conventional Commits format (chore:) preferred by AGENTS.md — consider chore: migrate workflows to Blacksmith runners.

Confidence Score: 5/5

Safe to merge — the only change is swapping the runner label for the syntax-check job to match what the e2e-signal-flow job already uses.

Both jobs now use the same pinned Blacksmith runner. The workflow logic, steps, and action versions are untouched, and the syntax-check job's single compile step has no system-level dependencies that would be affected by an Ubuntu version difference.

No files require special attention.

Important Files Changed

Filename Overview
.github/workflows/python-package-conda.yml Single-line runner migration for the syntax-check job from ubuntu-latest to blacksmith-4vcpu-ubuntu-2404; no logic changes.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A([push or pull_request to main]) --> B[syntax-check job]
    A --> C[e2e-signal-flow job]

    B --> B1["runs-on: blacksmith-4vcpu-ubuntu-2404\nwas: ubuntu-latest"]
    B1 --> B2[checkout]
    B2 --> B3[setup-python 3.11]
    B3 --> B4["python -m compileall money-machine/src-python"]

    C --> C1["runs-on: blacksmith-4vcpu-ubuntu-2404\nunchanged"]
    C1 --> C2[checkout]
    C2 --> C3[setup-python 3.11]
    C3 --> C4[pip install requirements + pytest]
    C4 --> C5[pytest e2e test_signal_to_execution_e2e]

    style B1 fill:#d4edda,stroke:#28a745
Loading

Reviews (1): Last reviewed commit: "Migrate workflows to Blacksmith" | Re-trigger Greptile

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