Skip to content

ci: Use macos-15-intel for building qis wheels#1248

Merged
aborgna-q merged 9 commits intomainfrom
ab/qis-wheels-macos-15
Nov 13, 2025
Merged

ci: Use macos-15-intel for building qis wheels#1248
aborgna-q merged 9 commits intomainfrom
ab/qis-wheels-macos-15

Conversation

@aborgna-q
Copy link
Copy Markdown
Collaborator

@aborgna-q aborgna-q commented Nov 11, 2025

Github started doing brownouts for macos-13 runners: https://github.com/CQCL/tket2/actions/runs/19271513759/job/55100761709

This PR bumps the runner version for intel-based mac builders.

drive-by: Rename the workflow in line with the other wheel builders
drive-by: Move the SHOULD_RUN check to avoid running on unrelated releases (we're already running on the merge commit), and cleanup file.
drive-by: Set MACOSX_DEPLOYMENT_TARGET = "15.0" for tket-c-api builds too

tket-py wheels run: https://github.com/CQCL/tket2/actions/runs/19333583104

@aborgna-q aborgna-q marked this pull request as ready for review November 11, 2025 17:07
@aborgna-q aborgna-q requested a review from a team as a code owner November 11, 2025 17:07
@aborgna-q aborgna-q requested a review from acl-cqc November 11, 2025 17:07
@aborgna-q aborgna-q force-pushed the ab/qis-wheels-macos-15 branch from 1a2ab65 to c8af0d6 Compare November 11, 2025 17:10
@codecov
Copy link
Copy Markdown

codecov bot commented Nov 12, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 79.11%. Comparing base (6bc3c4e) to head (c850c09).
⚠️ Report is 7 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1248      +/-   ##
==========================================
+ Coverage   79.06%   79.11%   +0.05%     
==========================================
  Files         160      160              
  Lines       20421    20413       -8     
  Branches    19489    19481       -8     
==========================================
+ Hits        16145    16150       +5     
+ Misses       3289     3280       -9     
+ Partials      987      983       -4     
Flag Coverage Δ
qis-compiler 100.00% <ø> (ø)
rust 78.45% <ø> (+0.05%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@aborgna-q aborgna-q requested a review from cqc-alec November 13, 2025 13:23
@aborgna-q aborgna-q added this pull request to the merge queue Nov 13, 2025
@aborgna-q aborgna-q removed this pull request from the merge queue due to a manual request Nov 13, 2025
@aborgna-q
Copy link
Copy Markdown
Collaborator Author

@cqc-alec
Copy link
Copy Markdown
Collaborator

Uhm

clang++: error: invalid version number in 'MACOSX_DEPLOYMENT_TARGET= "15.0"'

https://github.com/CQCL/tket2/actions/runs/19333066327/job/55300726160#step:8:733

Maybe you don't want the quotes?

@aborgna-q aborgna-q added this pull request to the merge queue Nov 13, 2025
Merged via the queue into main with commit d5b8dfc Nov 13, 2025
40 checks passed
@aborgna-q aborgna-q deleted the ab/qis-wheels-macos-15 branch November 13, 2025 14:16
github-merge-queue bot pushed a commit that referenced this pull request Nov 13, 2025
🤖 I have created a release *beep* *boop*
---


##
[0.12.11](tket-py-v0.12.10...tket-py-v0.12.11)
(2025-11-13)


### Bug Fixes

* **ci:** Make wheels compatible with MacOS 15.0
(#1248)
([49c5996](49c5996))
* Don't use opgroup in pytket barrier encoding
(#1251)
([49c5996](49c5996))
* guppy_to_circuit always returns num_operations = 0
(#1200)
([49c5996](49c5996))
* **pytket-decoder:** Avoid QAllocating and immediately freeing qubits
([#1256](#1256))
([228ff52](228ff52))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

---------

Co-authored-by: Agustín Borgna <[email protected]>
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