Skip to content

[ci] change macos intel platform to 12_0#53671

Merged
aslonnie merged 1 commit intomasterfrom
lonnie-250609-macup
Jun 10, 2025
Merged

[ci] change macos intel platform to 12_0#53671
aslonnie merged 1 commit intomasterfrom
lonnie-250609-macup

Conversation

@aslonnie
Copy link
Copy Markdown
Collaborator

@aslonnie aslonnie commented Jun 9, 2025

we upgraded the machine and releases

we upgraded the machine and releases

Signed-off-by: Lonnie Liu <lonnie@anyscale.com>
Copilot AI review requested due to automatic review settings June 9, 2025 19:50
@aslonnie aslonnie requested review from a team as code owners June 9, 2025 19:50
@aslonnie aslonnie requested a review from khluu June 9, 2025 19:50
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

This PR updates the macOS Intel wheel tag to use macosx_12_0_x86_64 following an upgrade and removes the old download script.

  • Remove the deprecated download_wheels.sh script
  • Update macOS wheel URLs in installation docs, tests, and automation library
  • Align CI automation to the new platform identifier

Reviewed Changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.

File Description
release/util/download_wheels.sh Removed outdated download script
doc/source/ray-overview/installation.rst Updated macOS x86_64 wheel URLs to use macosx_12_0
ci/ray_ci/automation/test_ray_wheels_lib.py Updated test fixtures to expect macosx_12_0_x86_64
ci/ray_ci/automation/ray_wheels_lib.py Replaced macosx_10_15_x86_64 with macosx_12_0_x86_64 in list
Comments suppressed due to low confidence (3)

release/util/download_wheels.sh:1

  • The download_wheels.sh script was removed; please verify and update any CI or documentation references to this script to avoid broken pipelines.
#!/usr/bin/env bash

doc/source/ray-overview/installation.rst:196

  • Bullet list items should be indented consistently in reStructuredText; align these lines with list indentation (e.g., prefix with two spaces).
* For MacOS x86_64, commits predating August 7, 2021 will have ``macosx_10_13`` in the filename instead of ``macosx_10_15``.

ci/ray_ci/automation/test_ray_wheels_lib.py:23

  • [nitpick] Tests currently only cover the Python 3.9 wheel variant for macosx_12_0; consider adding entries for other Python versions (3.10–3.13) to improve coverage.
"ray-1.0.0-cp39-cp39-macosx_12_0_x86_64",

@aslonnie aslonnie added the go add ONLY when ready to merge, run all tests label Jun 9, 2025
@aslonnie aslonnie merged commit 7243744 into master Jun 10, 2025
5 checks passed
@aslonnie aslonnie deleted the lonnie-250609-macup branch June 10, 2025 00:20
aslonnie added a commit that referenced this pull request Jun 13, 2025
mainly for docs

Signed-off-by: Lonnie Liu <lonnie@anyscale.com>
aslonnie added a commit that referenced this pull request Jun 13, 2025
mainly for docs update

Signed-off-by: Lonnie Liu <lonnie@anyscale.com>
elliot-barn pushed a commit that referenced this pull request Jun 18, 2025
we upgraded the machine and releases

Signed-off-by: Lonnie Liu <lonnie@anyscale.com>
Signed-off-by: elliot-barn <elliot.barnwell@anyscale.com>
elliot-barn pushed a commit that referenced this pull request Jul 2, 2025
we upgraded the machine and releases

Signed-off-by: Lonnie Liu <lonnie@anyscale.com>
Signed-off-by: elliot-barn <elliot.barnwell@anyscale.com>
weiquanlee pushed a commit to antgroup/ant-ray that referenced this pull request Aug 5, 2025
mainly for docs update

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

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