Skip to content

PATCHES.md: link the upstream vLLM PR for serve-model-path-match - #178

Merged
mhenrichsen merged 1 commit into
syv-ai:mainfrom
TyroneNel:patches-md-model-path-link
Sep 22, 2026
Merged

mhenrichsen merged 1 commit into
syv-ai:mainfrom
TyroneNel:patches-md-model-path-link

Conversation

@TyroneNel

Copy link
Copy Markdown
Contributor

One row, one cell.

serve-model-path-match says none yet under upstream, but the PR exists — vllm-project/vllm#58026, opened in the same batch as the #58025 and #58027 links the rows above and below it already carry. #167 was written before it was filed and the cell never caught up; that is my omission to fix.

The other twelve none yet rows are left alone: they are accurate.

The row landed with "none yet" because the vLLM PR was opened after syv-ai#167 was
written. It exists: vllm-project/vllm#58026, the same batch as the #58025 and
#58027 links the rows beside it already carry.
@mhenrichsen

Copy link
Copy Markdown
Contributor

Merged — and the omission is mine, not yours. #167's branch carried a commit that set this cell to vllm #58026; when I rebased it onto main after #166 landed, my conflict resolution on PATCHES.md kept the pre-link version of the row, and the squash went in without it. The other three rows from today (#58024, #58025, #58027) came through intact, and there are no duplicated rows, so this was the only casualty.

Confirmed the link points where it should: vllm-project/vllm#58026, "[Frontend] Accept the model path that /v1/models publishes", open, yours.

@mhenrichsen
mhenrichsen merged commit 13b30ea into syv-ai:main Sep 22, 2026
1 check passed
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