Skip to content

docs(changelog): note packaging extras removal in Unreleased - #433

Merged
ayushag-nv merged 1 commit into
NVIDIA-NeMo:mainfrom
LeonSGP43:docs/changelog-extras-removal
Aug 17, 2026
Merged

docs(changelog): note packaging extras removal in Unreleased#433
ayushag-nv merged 1 commit into
NVIDIA-NeMo:mainfrom
LeonSGP43:docs/changelog-extras-removal

Conversation

@LeonSGP43

@LeonSGP43 LeonSGP43 commented Aug 15, 2026

Copy link
Copy Markdown
Contributor

What

Adds a bullet to the Unreleased Removed section noting that the packaging extras [server], [gpu], and [all] were dropped together with the deprecated Python server stack — only [cli] remains.

Why

The 0.1.0 release notes advertise pip install nemo-switchyard with extras [server], [cli], [gpu], [all]. On current main, [project.optional-dependencies] defines only cli. The Unreleased section documents the server-stack removal but omits the extras removal, so users following the 0.1.0 packaging note get nemo-switchyard does not provide the extra 'server' warnings with silently missing dependencies.

Verification

# main
$ grep -A2 'optional-dependencies' pyproject.toml | grep -E '^(server|gpu|all|cli) =' 
cli = [
# v0.1.0 tag
$ gh api 'repos/NVIDIA-NeMo/Switchyard/contents/pyproject.toml?ref=v0.1.0' --jq .content | base64 -d | grep -E '^(server|gpu|all) ='
server = [

Signed-off-by: LeonSGP43 LeonSGP43@users.noreply.github.com

Summary by CodeRabbit

  • Documentation
    • Updated packaging documentation to reflect that only the CLI extra remains.
    • Documented installation of server functionality through the standalone switchyard-server binary.

The 0.1.0 release notes advertise 'pip install nemo-switchyard with
optional extras [server], [cli], [gpu], [all]'. The deprecated Python
server stack removal also dropped the [server], [gpu], and [all]
extras from pyproject.toml (only [cli] remains), but the Unreleased
Removed section does not mention it. Users following the 0.1.0
packaging note after upgrading hit 'does not provide the extra'
warnings. Document the removal.

Verified: pyproject.toml [project.optional-dependencies] on main
contains only 'cli'; the v0.1.0 tag contained server (and gpu/all).

Signed-off-by: LeonSGP43 <LeonSGP43@users.noreply.github.com>
@LeonSGP43
LeonSGP43 requested a review from a team as a code owner August 15, 2026 04:37
@coderabbitai

coderabbitai Bot commented Aug 15, 2026

Copy link
Copy Markdown

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: c0729112-c72c-4a8c-aaf9-997c0859d3cf

📥 Commits

Reviewing files that changed from the base of the PR and between 9ad6744 and acad01e.

📒 Files selected for processing (1)
  • CHANGELOG.md

Walkthrough

The changelog documents removal of the [server], [gpu], and [all] packaging extras. It directs users to install server functionality through the standalone switchyard-server binary.

Changes

Packaging documentation

Layer / File(s) Summary
Document packaging extra removal
CHANGELOG.md
The changelog records removal of the [server], [gpu], and [all] extras and identifies [cli] as the remaining extra. It directs users to the standalone switchyard-server binary.

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

Merge Risk: ⚪ Minimal · up to acad0

This is a localized changelog-only documentation change, and no actionable merge-blocking risk remains beyond normal checks and review.

Poem

I’m a rabbit with a changelog neat,
Three extras hop away on fleet feet.
[cli] stays in the burrow bright,
switchyard-server guides the night.
Documentation now grows clear—
No packaging carrots disappear!

🚥 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 describes the Unreleased changelog entry documenting packaging extras removal.
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.

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

@ayushag-nv
ayushag-nv enabled auto-merge (squash) August 17, 2026 16:56
@ayushag-nv
ayushag-nv merged commit f735d9d into NVIDIA-NeMo:main Aug 17, 2026
13 checks passed
@LeonSGP43

Copy link
Copy Markdown
Contributor Author

Thank you @ayushag-nv — glad this one landed as well. I'll keep an eye on #437 for any feedback.

michaelneale added a commit to michaelneale/Switchyard that referenced this pull request Aug 18, 2026
* origin/main: (31 commits)
  feat(server): add Dockerfile for switchyard-server container image (NVIDIA-NeMo#421)
  fix: normalize empty content-filter responses instead of raising (NVIDIA-NeMo#358)
  feat(python): unify LLM classifier bindings (NVIDIA-NeMo#465)
  feat(libsy): record task_kind and agent_role on the run span (NVIDIA-NeMo#249)
  fix(translation): accept SSE data fields with no space after the colon (NVIDIA-NeMo#447)
  fix(client): strip api-key and OpenAI org/project headers before forwarding (NVIDIA-NeMo#420)
  fix(llm-client): detect native sglang context-overflow messages (NVIDIA-NeMo#426)
  refactor(protocol): use typed HTTP status codes (NVIDIA-NeMo#457)
  fix(translation): preserve chat reasoning details (NVIDIA-NeMo#415)
  docs(changelog): note packaging extras removal in Unreleased (NVIDIA-NeMo#433)
  docs(changelog): fix broken Metrics Reference link (NVIDIA-NeMo#432)
  feat(libsy-llm-client): Move retry logic from libsy to libsy-llm-client (NVIDIA-NeMo#431)
  fix(server): use normalized session IDs in routing stats (NVIDIA-NeMo#430)
  feat(benchmark): pre-bake hermes agent into dataset images for closed-book runs (NVIDIA-NeMo#350)
  fix: support json_object classifier responses (NVIDIA-NeMo#411)
  fix(metrics): extend LLM latency histogram buckets (NVIDIA-NeMo#385)
  feat(server): forward inbound Anthropic auth (NVIDIA-NeMo#372)
  fix(protocol): normalize nested metadata strings (NVIDIA-NeMo#422)
  Revert "ci: base full CI on changed paths (NVIDIA-NeMo#403)" (NVIDIA-NeMo#414)
  feat: Move `Decision::reasoning` to a log message (NVIDIA-NeMo#413)
  ...

Co-authored-by: Michael Neale <michael.neale@gmail.com>
Signed-off-by: Michael Neale <michael.neale@gmail.com>

# Conflicts:
#	crates/switchyard-server/src/config.rs
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.

2 participants