Skip to content

Add @benchislett to codeowner for spec decode and structured outputs#24362

Merged
youkaichao merged 1 commit intovllm-project:mainfrom
CentML:benchislett-codeowner
Sep 6, 2025
Merged

Add @benchislett to codeowner for spec decode and structured outputs#24362
youkaichao merged 1 commit intovllm-project:mainfrom
CentML:benchislett-codeowner

Conversation

@benchislett
Copy link
Collaborator

@benchislett benchislett commented Sep 6, 2025

Thank you @youkaichao for the invitation to join as a vLLM committer. I'm grateful for the opportunity to participate in the development of vLLM as a cutting-edge serving engine, and I am excited to help maintain the speculative decoding and structured outputs features.

Special thanks to @WoosukKwon @mgoin @LucasWilkinson @njhill @aarnphm @comaniac @alexm-redhat @DarkLight1337 @ywang96 @robertgshaw2-redhat @russellb from the vLLM community for very helpful code reviews and feedback.

Signed-off-by: Benjamin Chislett <benjamin.chislett@centml.ai>
@mergify mergify bot added the ci/build label Sep 6, 2025
Copy link
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request adds @benchislett as a code owner for the speculative decoding and structured outputs features. The changes to the .github/CODEOWNERS file are correct and align with the intent of the pull request. I have no feedback as there are no critical or high-severity issues in this change.

Copy link
Member

@ywang96 ywang96 left a comment

Choose a reason for hiding this comment

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

Welcome! Will wait for @youkaichao to actually grant you write access from repo settings.

Copy link
Member

@youkaichao youkaichao left a comment

Choose a reason for hiding this comment

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

welcome!

@youkaichao youkaichao merged commit 0077c86 into vllm-project:main Sep 6, 2025
14 checks passed
eicherseiji pushed a commit to eicherseiji/vllm that referenced this pull request Sep 9, 2025
…llm-project#24362)

Signed-off-by: Benjamin Chislett <benjamin.chislett@centml.ai>
skyloevil pushed a commit to skyloevil/vllm that referenced this pull request Sep 13, 2025
…llm-project#24362)

Signed-off-by: Benjamin Chislett <benjamin.chislett@centml.ai>
FeiDaLI pushed a commit to FeiDaLI/vllm that referenced this pull request Sep 25, 2025
…llm-project#24362)

Signed-off-by: Benjamin Chislett <benjamin.chislett@centml.ai>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants