Skip to content

Conversation

ScottWillcox
Copy link
Contributor

@ScottWillcox ScottWillcox commented Oct 22, 2025

note: this does not encompass fixing the builds for MacOS. that will be done in GRIDEDIT-1971.

  • removed MacOS-13 runner which will be removed in the near future.
  • added the MacOS-15 runner. This is the arm64 based runner.

Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR updates GitHub Actions workflow configurations to replace the deprecated macOS-13 runner with macOS-15, aligning the CI/CD pipeline with newer macOS runner versions.

Key Changes:

  • Removed macos-13 runner from the build matrix in feature/master and release workflows
  • Added macos-15 runner to the build matrix
  • Updated the Xcode version selection condition to include macos-14 and macos-15 runners

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
.github/workflows/build-and-test-workflow.yml Updated Xcode version selection condition to support macos-14 and macos-15 runners
.github/workflows/build-and-test-release.yml Replaced macos-13 runner with macos-15 in the build matrix
.github/workflows/build-and-test-feature-master.yml Replaced macos-13 runner with macos-15 in the build matrix

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.


- name: Set Xcode version
if: inputs.platform == 'macos-13' || inputs.platform == 'macos-13-xlarge'
if: inputs.platform == 'macos-13-xlarge' || inputs.platform == 'macos-14' || inputs.platform == 'macos-15'
Copy link
Collaborator

Choose a reason for hiding this comment

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

If setting of the xcode version to 14.1 gets the macos 14 and 15 builds to complete then I guess this is good.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

this was already in there. just updated the checks to the correct version. this might change as part of GRIDEDIT-1971. but I'll see that when I get there.

@ScottWillcox ScottWillcox requested review from RStolkerDeltares and removed request for ahmad-el-sayed October 22, 2025 12:08

- name: Set Xcode version
if: inputs.platform == 'macos-13' || inputs.platform == 'macos-13-xlarge'
if: inputs.platform == 'macos-14' || inputs.platform == 'macos-15'

Choose a reason for hiding this comment

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

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.

3 participants