Skip to content

docs(vllm): drop the SSE streaming branch pin now that PR 25 is merged - #37

Merged
rickstaa merged 1 commit into
mainfrom
rs/vllm-streaming-mainline
Jul 28, 2026
Merged

docs(vllm): drop the SSE streaming branch pin now that PR 25 is merged#37
rickstaa merged 1 commit into
mainfrom
rs/vllm-streaming-mainline

Conversation

@rickstaa

@rickstaa rickstaa commented Jul 28, 2026

Copy link
Copy Markdown
Member

Gateway PR livepeer/livepeer-python-gateway#25 (SSE streaming) merged into ja/live-runner on 2026-07-27, so the interim branch pin is no longer needed.

  • vllm/pyproject.toml: point the livepeer-gateway uv source at ja/live-runner instead of rs/live-runner-streaming
  • vllm/README.md: remove the IMPORTANT note about the unmerged PR from the streaming section
  • README.md: remove the matching note from the repo README

Verified locally with uv lock --upgrade-package livepeer-gateway && uv sync (resolves to 9f2bc20 on ja/live-runner).

Gateway PR 25 landed on ja/live-runner, so the vllm example no longer
needs the rs/live-runner-streaming pin. Point the SDK source at the
mainline branch and remove the interim notes from both READMEs.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings July 28, 2026 08:02

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Removes the temporary documentation and dependency pinning that referenced the pre-merge SSE streaming work in the livepeer-gateway SDK, now that gateway PR #25 has landed on ja/live-runner.

Changes:

  • Update the livepeer-gateway uv git source in vllm/pyproject.toml to track ja/live-runner instead of the interim rs/live-runner-streaming branch.
  • Remove “IMPORTANT” notes in both the repo README and vllm README that warned SSE streaming required the unmerged PR/branch.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
vllm/README.md Removes the outdated SSE streaming warning block now that streaming is available on the main gateway branch used by this repo.
vllm/pyproject.toml Switches the livepeer-gateway git branch source to ja/live-runner.
README.md Removes the repo-level SSE streaming warning that referenced the now-merged gateway PR/branch pin.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@rickstaa
rickstaa merged commit 07a08a1 into main Jul 28, 2026
2 checks passed
@rickstaa
rickstaa deleted the rs/vllm-streaming-mainline branch July 28, 2026 08:13
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