Skip to content

Conversation

seemewalkin
Copy link
Contributor

@seemewalkin seemewalkin commented Aug 30, 2025

Summary

Should improve speed of release workflow by switching the initial job from github-ubuntu runner to faster Blacksmith runner.

Related Linear tickets, Github issues, and Community forum posts

https://linear.app/n8n/issue/CAT-1280/switch-from-github-ubuntu-to-blacksmith-runner-in-the-release-job

Review / Merge checklist

  • PR title and summary are descriptive. (conventions)
  • Docs updated or follow-up ticket created.
  • Tests included.
  • PR Labeled with release/backport (if the PR is an urgent fix that needs to be backported)

@seemewalkin seemewalkin added the team:cats Issue is with the Cats team label Aug 30, 2025
@seemewalkin seemewalkin changed the title ci: Change runner to blacksmith on publish to npm job ci: Change runner to blacksmith on publish to npm release job Aug 30, 2025
Copy link
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

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

1 issue found across 1 file

React with 👍 or 👎 to teach cubic. You can also tag @cubic-dev-ai to give feedback, ask questions, or re-run the review.

@seemewalkin seemewalkin removed the request for review from shortstacked August 30, 2025 14:19
Copy link

codecov bot commented Aug 30, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@n8n-assistant n8n-assistant bot added the n8n team Authored by the n8n team label Aug 30, 2025
@shortstacked
Copy link
Contributor

You could look at swapping it out for the reusable action we have that does all the node setup in one step.
That would take care of using the blacksmith cache as well. It's set here to still use the GH cache which means we get faster build times but still slower cache speeds.
If we swap to using the blacksmith cache as well we should be able to get cache hits from earlier stages of building/testing too.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
n8n team Authored by the n8n team team:cats Issue is with the Cats team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants