Skip to content

Remove macos-13#299

Merged
Cadair merged 3 commits into
OpenAstronomy:mainfrom
Cadair:remove-macos-13
Jul 29, 2025
Merged

Remove macos-13#299
Cadair merged 3 commits into
OpenAstronomy:mainfrom
Cadair:remove-macos-13

Conversation

@Cadair
Copy link
Copy Markdown
Member

@Cadair Cadair commented Jul 28, 2025

Fixes #298

Comment thread .github/workflows/test_publish.yml Outdated
Comment thread tools/tox_matrix.py
# if Python is <3.10 we can't use macos-latest which is arm64
try:
if Version(item["python_version"]) < Version('3.10') and item["os"] == "macos-latest":
item["os"] = "macos-13"
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I thought about not removing this for a while, but actually I think removing it and having the error be "can't find Python 3.9" is better than "huh what why is this on macos-13".

Copy link
Copy Markdown

@dhomeier dhomeier Jul 28, 2025

Choose a reason for hiding this comment

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

But 3.9 is still available on macos-14, right? (or rather, added in actions/runner-images#9813)

Copy link
Copy Markdown

@dhomeier dhomeier Jul 28, 2025

Choose a reason for hiding this comment

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

Ah, forgot again that that is the macos-large version. But then there won't actually be a free version available with Python < '3.11' anymore, unless I am missing something? Or they are yet adding it to macos-14 and/or -15; support policy indicates that 3.10 should still be available after 3.14 is out, though it does not explicitly guarantee this for all platforms...

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

🤷 lol

Comment thread docs/source/publish.rst
targets: ''

For additional configuration extra arguments can be passed by making a target a dictionary.
An example of this is specifying the runner for a target, such as building macos x86_64 wheels on native x86_64 runners:
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Anyone got an idea of an example here which would still work?

Copy link
Copy Markdown
Contributor

@neutrinoceros neutrinoceros left a comment

Choose a reason for hiding this comment

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

minor suggestion, otherwise LGTM

Comment thread .github/workflows/test_publish.yml
Comment thread docs/source/publish.rst

targets:
- target: cp311-macosx_x86_64
runs-on: macos-13
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Do you want to swap out the example instead of removing it altogether?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Preferably

@Cadair Cadair force-pushed the remove-macos-13 branch from 1eae87a to 83e3307 Compare July 29, 2025 14:55
@Cadair Cadair merged commit 828af55 into OpenAstronomy:main Jul 29, 2025
75 checks passed
@Cadair Cadair deleted the remove-macos-13 branch July 29, 2025 15:12
@bsipocz
Copy link
Copy Markdown
Member

bsipocz commented Aug 1, 2025

The bigger issue I think is the arm non-arm switch, and that's what we run into regarding this upgrade for pytest-arraydiff (I mean the bug is downstream as the tests should handle the missing dependency, yet here we are): astropy/pytest-arraydiff#55

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.

BLD: macos-13 going away

5 participants