Skip to content

Bump macos runner version#7012

Closed
da-woods wants to merge 5 commits intocython:masterfrom
da-woods:bump-os-runner-version
Closed

Bump macos runner version#7012
da-woods wants to merge 5 commits intocython:masterfrom
da-woods:bump-os-runner-version

Conversation

@da-woods
Copy link
Contributor

@da-woods
Copy link
Contributor Author

image

I think something in the project setting explicitly requires some runners called macos-13 to complete, so I may not be able to merge this...

@da-woods
Copy link
Contributor Author

There's a bit of a mess to sort out here with line_profiler (including that we're not running the test on Python >= 3.10)

@da-woods da-woods marked this pull request as draft July 13, 2025 12:05
@da-woods
Copy link
Contributor Author

Doesn't immediately look like this is going to work. I'll close this PR for now and come back to it some other time

@da-woods da-woods closed this Jul 13, 2025
@scoder
Copy link
Contributor

scoder commented Jul 13, 2025 via email

@da-woods
Copy link
Contributor Author

Currently we're blocked by the line_profiler tests. Updating to the newer OS X image means that line_profile needs to compile manually and the old version we rely on doesn't build with recent Cython (and doesn't have a suitably restrictive version pin). I'm sure it's fixable, but it's probably not worth the effort.

It looks like line_profile have very recently fixed their Cython compatibility (pyutils/line_profiler#352) so as soon as that makes into into a release we should be able to use that.

So I think we just wait for that to happen before changing any OS X runners (at which point I'll need you to change the explicit list of jobs)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants