Skip to content

[Bugfix] Python 3.10 compatibility for Self#27918

Merged
DarkLight1337 merged 1 commit intovllm-project:mainfrom
DarkLight1337:fix-py310
Nov 1, 2025
Merged

[Bugfix] Python 3.10 compatibility for Self#27918
DarkLight1337 merged 1 commit intovllm-project:mainfrom
DarkLight1337:fix-py310

Conversation

@DarkLight1337
Copy link
Copy Markdown
Member

@DarkLight1337 DarkLight1337 commented Nov 1, 2025

Purpose

FIX #26519 (comment)

Test Plan

Test Result


Essential Elements of an Effective PR Description Checklist
  • The purpose of the PR, such as "Fix some issue (link existing issues this PR will resolve)".
  • The test plan, such as providing test command.
  • The test results, such as pasting the results comparison before and after, or e2e results
  • (Optional) The necessary documentation update, such as updating supported_models.md and examples for a new model.
  • (Optional) Release notes update. If your change is user facing, please update the release notes draft in the Google Doc.

Signed-off-by: DarkLight1337 <tlleungac@connect.ust.hk>
@DarkLight1337 DarkLight1337 added the ready ONLY add when PR is ready to merge/full CI is needed label Nov 1, 2025
Copy link
Copy Markdown
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 correctly addresses a Python 3.10 compatibility issue by replacing typing.Self with typing_extensions.Self. Since typing.Self is only available in Python 3.11 and later, this change ensures backward compatibility. The modification is accurate and well-contained within vllm/config/structured_outputs.py. I find no issues with this change.

@DarkLight1337 DarkLight1337 enabled auto-merge (squash) November 1, 2025 14:11
@DarkLight1337 DarkLight1337 merged commit 99d69af into vllm-project:main Nov 1, 2025
51 checks passed
@DarkLight1337 DarkLight1337 deleted the fix-py310 branch November 1, 2025 15:28
ZhengHongming888 pushed a commit to ZhengHongming888/vllm that referenced this pull request Nov 8, 2025
Signed-off-by: DarkLight1337 <tlleungac@connect.ust.hk>
rtourgeman pushed a commit to rtourgeman/vllm that referenced this pull request Nov 10, 2025
Signed-off-by: DarkLight1337 <tlleungac@connect.ust.hk>
devpatelio pushed a commit to SumanthRH/vllm that referenced this pull request Nov 29, 2025
Signed-off-by: DarkLight1337 <tlleungac@connect.ust.hk>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready ONLY add when PR is ready to merge/full CI is needed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants