Skip to content

Fix Python version mismatch in setup.py#1025

Merged
DefTruth merged 1 commit into
vipshop:mainfrom
FNGarvin:main
May 28, 2026
Merged

Fix Python version mismatch in setup.py#1025
DefTruth merged 1 commit into
vipshop:mainfrom
FNGarvin:main

Conversation

@FNGarvin
Copy link
Copy Markdown
Contributor

Greetings DefTruth,

Thanks for your wonderful work.

pyproject.toml and the project documentation state that Python >=3.10 is supported. However, setup.py hardcodes python_requires=">=3.12", which allows things like building cp310 wheels that won't install in cp310 environments.

This trivial PR updates setup.py to match pyproject.toml.

Best regards,
FNG

Align setup.py with pyproject.toml and project documentation 
by lowering the required Python version to >=3.10.  This fixes 
wheel compilation failures on Python 3.10 environments.
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates packaging metadata so setup.py matches the project’s declared minimum supported Python version (>=3.10) in pyproject.toml, preventing wheels from advertising compatibility with Python versions they won’t install on.

Changes:

  • Set python_requires in setup.py from >=3.12 to >=3.10 to align with pyproject.toml.

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

Copy link
Copy Markdown
Member

@DefTruth DefTruth left a comment

Choose a reason for hiding this comment

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

LGTM, Thanks for this fix~

@DefTruth DefTruth merged commit 4a1bcce into vipshop:main May 28, 2026
4 checks 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.

3 participants