Skip to content

[ci] upgrade twine and packaging#59568

Merged
aslonnie merged 1 commit intomasterfrom
lonnie-251218-twineup
Dec 19, 2025
Merged

[ci] upgrade twine and packaging#59568
aslonnie merged 1 commit intomasterfrom
lonnie-251218-twineup

Conversation

@aslonnie
Copy link
Collaborator

otherwise the wheel uploading is failing with older versions.

@aslonnie aslonnie requested a review from a team December 19, 2025 06:12
otherwise the wheel uploading will be failing

Signed-off-by: Lonnie Liu <lonnie@anyscale.com>
@aslonnie aslonnie force-pushed the lonnie-251218-twineup branch from 69b27ea to 9f14cd4 Compare December 19, 2025 06:13
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 updates the twine dependency from version 6.1.0 to 6.2.0 and the packaging dependency from version 24.0 to 25.0. These changes are reflected in both release/requirements_py310.in and release/requirements_py310.txt, including the corresponding SHA256 hash updates for the locked dependencies. No review comments were provided.

"PyGithub",
"requests",
"twine == 6.1.0",
"twine == 6.2.0",
Copy link

Choose a reason for hiding this comment

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

Bug: twine version not updated in requirements_buildkite.in

The twine version was updated to 6.2.0 in setup.py, requirements_py310.in, and requirements_py310.txt, but requirements_buildkite.in still has twine == 6.1.0. The comment in setup.py at line 16 explicitly states "Keep this in sync with requirements_buildkite.in", so this creates an inconsistency that violates the documented sync requirement. This could cause different CI environments to use different twine versions.

Fix in Cursor Fix in Web

@ray-gardener ray-gardener bot added the devprod label Dec 19, 2025
@aslonnie aslonnie added the go add ONLY when ready to merge, run all tests label Dec 19, 2025
@aslonnie aslonnie merged commit e19d4d1 into master Dec 19, 2025
7 checks passed
@aslonnie aslonnie deleted the lonnie-251218-twineup branch December 19, 2025 13:45
Yicheng-Lu-llll pushed a commit to Yicheng-Lu-llll/ray that referenced this pull request Dec 22, 2025
otherwise the wheel uploading is failing with older versions.

Signed-off-by: Lonnie Liu <lonnie@anyscale.com>
AYou0207 pushed a commit to AYou0207/ray that referenced this pull request Jan 13, 2026
otherwise the wheel uploading is failing with older versions.

Signed-off-by: Lonnie Liu <lonnie@anyscale.com>
Signed-off-by: jasonwrwang <jasonwrwang@tencent.com>
lee1258561 pushed a commit to pinterest/ray that referenced this pull request Feb 3, 2026
otherwise the wheel uploading is failing with older versions.

Signed-off-by: Lonnie Liu <lonnie@anyscale.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

devprod go add ONLY when ready to merge, run all tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants