Skip to content

chore(ci): update gh runners#130

Open
JP-Ellis wants to merge 1 commit into
surpher:feat/v2.0.0from
JP-Ellis:chore/update-gh-runners
Open

chore(ci): update gh runners#130
JP-Ellis wants to merge 1 commit into
surpher:feat/v2.0.0from
JP-Ellis:chore/update-gh-runners

Conversation

@JP-Ellis

Copy link
Copy Markdown

📝 Summary of Changes

Update the runners to macos15 and macos-15-intel, testing against both xcode 16 and 26 (identifying the latest minor version available).

The older macos-13 runner is not longer available, and macos-14 will be deprecated soon-ish.

⚠️ Items of Note

I have cleaned up a bit the matrix, and now infer the scheme and destination based on the platform.

🧐🗒 Reviewer Notes

Once merged, I will rebase #129 to check that CI passes.

🔨 How To Test

Run CI :)

Update the runners to macos15 and macos-15-intel, testing against both
xcode 16 and 26 (identifying the latest minor version available).

The older macos-13 runner is not longer available, and macos-14 will be
deprecated soon-ish.

Signed-off-by: JP-Ellis <josh@jpellis.me>
@JP-Ellis

Copy link
Copy Markdown
Author

Hmm, there certainly seem to be some issues. Maybe upgrading won't be as straightforward 🤔

@surpher

surpher commented Jul 16, 2026

Copy link
Copy Markdown
Owner

@JP-Ellis I've updated the CI config so it's building on latest-ish systems with f9bce99

@surpher

surpher commented Jul 16, 2026

Copy link
Copy Markdown
Owner

Feel free to submit any potential improvements. 👍

@surpher

surpher commented Jul 16, 2026

Copy link
Copy Markdown
Owner

One I can think of of the top of my head is setting env: DEVELOPER_DIR instead of running a sudo script on xcode-select.

env:
  DEVELOPER_DIR: /Applications/Xcode-${{ matrix.xcode }}.app/Contents/Developer

If that is how Xcode applications are named on GH hosted runners.
Or use a step from marketplace 🤷

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