Skip to content

fixup: use large runner in publish to make include-output-paths faster#497

Closed
cole-h wants to merge 1 commit into
mainfrom
faster-upload-output-paths
Closed

fixup: use large runner in publish to make include-output-paths faster#497
cole-h wants to merge 1 commit into
mainfrom
faster-upload-output-paths

Conversation

@cole-h

@cole-h cole-h commented Jun 12, 2026

Copy link
Copy Markdown
Member

Motivation

Context

Summary by CodeRabbit

  • Chores
    • Updated CI/CD pipeline infrastructure configuration to optimize build performance.

@cole-h cole-h enabled auto-merge June 12, 2026 01:12
@coderabbitai

coderabbitai Bot commented Jun 12, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 8f5bea64-548a-4290-b46e-3bb37de3f637

📥 Commits

Reviewing files that changed from the base of the PR and between 2a5c930 and f4382b9.

📒 Files selected for processing (1)
  • .github/workflows/ci.yml

📝 Walkthrough

Walkthrough

The CI workflow is updated to run the publish job on a higher-specification runner. The publish job's runs-on field changes from ubuntu-latest to UbuntuLatest32Cores128G, providing more compute resources for the release process.

Changes

CI Publish Runner Upgrade

Layer / File(s) Summary
Publish job runner specification
.github/workflows/ci.yml
The publish job is now configured to run on UbuntuLatest32Cores128G instead of ubuntu-latest, allocating a dedicated higher-specification runner for the release process.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Suggested reviewers

  • grahamc

Poem

🐰 A workflow so fleet,
Now runs on hardware complete,
Thirty-two cores run deep,
While publishing keeps
Release cycles neat!

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and specifically describes the main change: switching to a large runner for the publish job to improve performance of include-output-paths.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch faster-upload-output-paths

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions

Copy link
Copy Markdown

@cole-h cole-h disabled auto-merge June 12, 2026 01:20
@cole-h cole-h closed this Jun 12, 2026
@cole-h cole-h deleted the faster-upload-output-paths branch June 12, 2026 01:20
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