Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bump Xcode version to 14.3.1 #127

Merged
merged 1 commit into from
Oct 2, 2023
Merged

Conversation

kwridan
Copy link
Contributor

@kwridan kwridan commented Oct 2, 2023

Describe your changes

  • Bumped github workflow to build with macOS 13 images + Xcode 14.3.1
  • Updated minimum OS to be 10.15 (as that is the minimum by some of the newer versions of the dependencies - e.g. SwiftToolsSupport)

Testing performed

  • Verify CI passes

marciniwanicki
marciniwanicki previously approved these changes Oct 2, 2023
@kwridan
Copy link
Contributor Author

kwridan commented Oct 2, 2023

We might be hitting actions/runner-images#8341

 make build
  shell: /bin/bash -e {0}
  env:
    DEVELOPER_DIR: /Applications/Xcode_14.3.1.app/Contents/Developer
xcrun: error: missing DEVELOPER_DIR path: /Applications/Xcode_14.3.1.app/Contents/Developer

- Bumped github workflow to build with macOS 13 images + Xcode 14.3.1
- Updated minimum OS to be 10.15 (as that is the minimum by some of the newer versions of the dependencies - e.g. SwiftToolsSupport)

Test Plan:

- Verify CI passes

Signed-off-by: Kassem Wridan <kwridan@bloomberg.net>
@kwridan
Copy link
Contributor Author

kwridan commented Oct 2, 2023

My mistake, there were 2 steps that needed an update with the runner 🤦‍♂️

@codecov
Copy link

codecov bot commented Oct 2, 2023

Codecov Report

All modified lines are covered by tests ✅

Comparison is base (a2c5476) 96.71% compared to head (fd394e5) 96.95%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #127      +/-   ##
==========================================
+ Coverage   96.71%   96.95%   +0.24%     
==========================================
  Files          50       50              
  Lines        2773     2894     +121     
==========================================
+ Hits         2682     2806     +124     
+ Misses         91       88       -3     

see 23 files with indirect coverage changes

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

@kwridan kwridan marked this pull request as ready for review October 2, 2023 11:50
@kwridan kwridan merged commit 9baa4dd into bloomberg:main Oct 2, 2023
5 checks passed
@kwridan kwridan deleted the update/xcode-14-3-1 branch October 2, 2023 11:51
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.

None yet

2 participants