Skip to content

ci(release): publish switchyard-runner - #614

Merged
nachiketb-nvidia merged 1 commit into
mainfrom
bbednarski/publish-switchyard-runner
Sep 3, 2026
Merged

ci(release): publish switchyard-runner#614
nachiketb-nvidia merged 1 commit into
mainfrom
bbednarski/publish-switchyard-runner

Conversation

@bbednarski9

@bbednarski9 bbednarski9 commented Sep 3, 2026

Copy link
Copy Markdown
Contributor

Summary

  • publish switchyard-runner to crates.io on tagged releases
  • validate its version alongside the other released Rust crates
  • document the dependency-ordered release sequence

Validation

  • cargo package --allow-dirty --list --package switchyard-runner
  • YAML parse of .github/workflows/publish.yml
  • git diff --check

Summary by CodeRabbit

  • Chores

    • Updated the release process to validate and publish the runner package before the server package.
    • Expanded release checks to cover all published crates.
  • Documentation

    • Updated release documentation to reflect the current crate publication order.
    • Clarified that the registry token must have permission to publish all six crates.

Signed-off-by: Bryan Bednarski <bbednarski@nvidia.com>
@bbednarski9
bbednarski9 requested a review from a team as a code owner September 3, 2026 19:01
@github-actions

github-actions Bot commented Sep 3, 2026

Copy link
Copy Markdown
PR Preview Action v1.8.1

🚀 View preview at
https://NVIDIA-NeMo.github.io/Switchyard/pr-preview/pr-614/

Built to branch gh-pages at 2026-09-03 19:02 UTC.
Preview will be ready when the GitHub Pages deployment is complete.

@coderabbitai

coderabbitai Bot commented Sep 3, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Enterprise

Run ID: 90c0cd04-6cd0-4c33-9eeb-50f767ba6821

📥 Commits

Reviewing files that changed from the base of the PR and between bb07f99 and ca81dd9.

📒 Files selected for processing (2)
  • .github/workflows/publish.yml
  • docs/internal/release_workflow.md

Included review availability: Your plan provides up to 12 included reviews per hour; 11 remain after this review.


Walkthrough

The Rust release workflow now validates and publishes switchyard-runner. Release documentation now lists six crates, their publication order, and the required token scope.

Changes

Rust crate release workflow

Layer / File(s) Summary
Validate, publish, and document crate order
.github/workflows/publish.yml, docs/internal/release_workflow.md
The workflow validates switchyard-runner and publishes it before switchyard-server. The documentation places switchyard-translation before switchyard-libsy, adds switchyard-runner, and updates token requirements for six crates.

Estimated code review effort: 1 (Trivial) | ~5 minutes

Merge Risk: ⚪ Minimal · up to ca81d

Tagged releases now validate and publish switchyard-runner in the documented dependency order, with matching token guidance. No current merge-blocking risk is identified.

Poem

A rabbit checks the crate release queue
Runner hops in where it should
Six crates march in order
The token holds the keys
Server follows with a gentle thump

🚥 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 concisely identifies the main change: publishing switchyard-runner in the release workflow.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
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.
Full details: Docstring Coverage

Explanation

No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files. (2 skipped: 2 unsupported.)


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

@nachiketb-nvidia
nachiketb-nvidia merged commit e6bced8 into main Sep 3, 2026
20 checks passed
@nachiketb-nvidia
nachiketb-nvidia deleted the bbednarski/publish-switchyard-runner branch September 3, 2026 19:29
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.

3 participants