Thanks to visit codestin.com
Credit goes to github.com

Skip to content

ci: add macOS 26 target#2937

Merged
madrob merged 2 commits intoml-explore:mainfrom
madrob:macos-26
Dec 22, 2025
Merged

ci: add macOS 26 target#2937
madrob merged 2 commits intoml-explore:mainfrom
madrob:macos-26

Conversation

@madrob
Copy link
Contributor

@madrob madrob commented Dec 19, 2025

Proposed changes

Add macOS 26 target to all GitHub build targets

Checklist

Put an x in the boxes that apply.

  • I have read the CONTRIBUTING document
  • I have run pre-commit run --all-files to format my code / installed pre-commit prior to committing changes
  • I have added tests that prove my fix is effective or that my feature works
  • I have updated the necessary documentation (if needed)

@madrob madrob requested review from angeloskath and awni December 19, 2025 21:20
Comment on lines +118 to +122
- name: Build macOS 26 package
uses: ./.github/actions/build-macos-release
with:
macos-target: 26.0
build-backend: ${{ matrix.python-version == '3.10' }}
Copy link
Member

@awni awni Dec 19, 2025

Choose a reason for hiding this comment

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

We will do the 26.0 release manually for now until we can get on macOS 26.2 (which is needed to build neural accelerator and RDMA support).

So please remove that part.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

To explicitly confirm, we'll still leave it for both the nightly and the PR builds, yes?

Copy link
Member

Choose a reason for hiding this comment

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

Yes I think it's great to start testing 26.0 👍

For the nightly it's fine.. the only downside is burning compute.. but it's also not necessary so we can remove it and add it back once we do the 26.0 release in CI.

Copy link
Member

@awni awni left a comment

Choose a reason for hiding this comment

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

🙏

@madrob madrob merged commit 5205de5 into ml-explore:main Dec 22, 2025
15 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.

2 participants