Skip to content

[release-automation] Add option to add build tag when uploading wheels to pypi#51517

Merged
aslonnie merged 3 commits intomasterfrom
khluu/post1_wheels
Mar 21, 2025
Merged

[release-automation] Add option to add build tag when uploading wheels to pypi#51517
aslonnie merged 3 commits intomasterfrom
khluu/post1_wheels

Conversation

@khluu
Copy link
Contributor

@khluu khluu commented Mar 19, 2025

  • Add helper function to add build tag (e.g. -1) right after Ray version in the wheel name, in cases where original wheels uploaded to pypi/test pypi are corrupted.

p
Signed-off-by: kevin <kevin@anyscale.com>
@khluu khluu requested a review from a team as a code owner March 19, 2025 08:26
@khluu khluu requested a review from aslonnie March 19, 2025 08:27
p
Signed-off-by: kevin <kevin@anyscale.com>
commit_hash: The commit hash of the green commit.
ray_version: The version of Ray.
directory_path: The directory to download the wheels to.
build_tag: The build tag to add to the downloaded wheels.
Copy link
Collaborator

Choose a reason for hiding this comment

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

where is this arg? wrong place?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

oh yes it used to be there before I moved it.. but I forgot to delete the docstring

Copy link
Collaborator

@aslonnie aslonnie left a comment

Choose a reason for hiding this comment

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

nice!

could you fix the lints?

Signed-off-by: kevin <kevin@anyscale.com>
@khluu khluu requested a review from aslonnie March 21, 2025 02:59
@aslonnie aslonnie added the go add ONLY when ready to merge, run all tests label Mar 21, 2025
@aslonnie aslonnie self-requested a review March 21, 2025 16:00
@aslonnie aslonnie merged commit 41e3b38 into master Mar 21, 2025
6 checks passed
@aslonnie aslonnie deleted the khluu/post1_wheels branch March 21, 2025 22:32
dhakshin32 pushed a commit to dhakshin32/ray that referenced this pull request Mar 27, 2025
…s to pypi (ray-project#51517)

- Add helper function to add build tag (e.g. `-1`) right after Ray
version in the wheel name, in cases where original wheels uploaded to
pypi/test pypi are corrupted.

---------

Signed-off-by: kevin <kevin@anyscale.com>
Signed-off-by: Dhakshin Suriakannu <d_suriakannu@apple.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

community-backlog go add ONLY when ready to merge, run all tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

Comments